Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Repair broken links, and some files shouldn't exist #468

Merged
merged 1 commit into from
Feb 9, 2024

Conversation

Fangliding
Copy link
Member

删掉的都是vscode产生的配置文件
顺便帮助我们改善此界面坏掉了 因为本来好好的配置被放到了locales下面(¿)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

建议, 有一个项目级别的tabSize设置, 防止后来的维护者或者开发者, 在本地保存的时候, 可能会被强制格式化为4个空格, 2个空格, 或者一个tab.

项目级别的tabSize设置, 可以使得各个角色的代码一致性更好.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

没见过搞这个的
而且也不是通过一个.vscode啊

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

已经有 workflow 做这个了

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Fangliding
Copy link
Member Author

Fangliding commented Feb 6, 2024

再ping一下
我读了一下那个 stackoverflow 有说加的有说不建议的 多数回复都是不建议加 支持的主要是原因是有必要在团队内共享的设置 一些与环境有关的配置 tabsize我觉得不算 不影响执行反正js也不是强制缩进 摸过这些代码部分的人也就个位数 仓库的主要部分还是docs 要format一遍动动手的事 而且确实是意外add进来的 到时候搞不好还要被隔壁笑话

抛开这些 这里最主要是fix一个问题 比如说 https://xtls.github.io/config/#%E5%9F%BA%E7%A1%80%E9%85%8D%E7%BD%AE%E6%A8%A1%E5%9D%97 下面的帮助我们改善此界面直接跳到 https://github.com/xtls/xray-core/edit/main/docs/config/README.md 去了 这个pr主要是处理这个问题 几天前坏到现在

@yuhan6665
Copy link
Member

确实都可以 既然 @aoleou 已经是文档管理员了 按他的意思为准
修复的部分没问题

@Fangliding
Copy link
Member Author

wtf

@yuhan6665
Copy link
Member

我工作中确实遇到过同事给我们详细介绍了 .ide 文件夹里有些文件不能check in 有些则应该check in 然后还在CI 里面做了check in 之前的各种格式,commit message 的检查
虽然我个人觉得没必要 但是还是要多看别人的长处

@aoleou
Copy link
Contributor

aoleou commented Feb 9, 2024

@Fangliding

我非常欣赏风扇哥的执着. 说一下我的结论: 这个 Xray-docs-next 项目, 最核心的资产是文档内容. 那些基础配置, 都是复制粘贴过来的. 既然风扇哥,想合并,那就合并好了, 后面我得空了, 简单加上husky也一样可以实现我想要的需求, 而且更好.

@yuhan6665
感谢支持, 还记得, 第一次折腾这个事情, 就是从yuhan6665 的那一句 大佬, 开始的, 现在离成为大佬还有很长的路要走.

@yuhan6665 yuhan6665 merged commit 4cf6e98 into XTLS:main Feb 9, 2024
1 check passed
@yuhan6665
Copy link
Member

好的 希望这个项目能给大家在专业/兴趣中也获得一些意想不到的收获

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants