reset settings to avoid auto-format#239
Merged
ccmywish merged 2 commits intoRubyMetric:devfrom Aug 7, 2025
Merged
Conversation
|
Hi @Mikachu2333, ❤️ 感谢你的贡献!我们将在5天内阅读此 PR 并回复你 |
ccmywish
requested changes
Aug 7, 2025
Contributor
There was a problem hiding this comment.
👍👍👍
"editor.tabSize": 2,这一项应当删除,有这么几个原因:
- 和
.editorconfig可能有冲突 Makefile对 tab 很敏感,我不知道设置成这个是否会影响Makefile- 你可以看一下
.editorconfig这个文件,JSON是设置为4个空格的,这是因为 VS Code 经常会强制重写.vscode里的文件,总是使用4个空格 - 项目里还有
Perl代码,实际上按照Perl的标准应该用4个空格缩进,只不过我一直都没有调整这一点
我觉得还是集中用 .editorconfig 里的设置来控制合适一些。
我觉得整个项目应该采用 入乡随俗 的原则:官方有比较唯一性推荐的,沿用官方。比如 YAML 官方推荐用几个空格(2个),我们尽量用几个,JSON 官方推荐用几个空格(4个),我们就用几个。具体某个编程语言,如 Perl 这样的,虽然有不同的风格,但是 4 个空格是 Larry Wall 所认同的,应当按照4个。
所以应当把这个配置项删除。
Collaborator
Author
感觉是得改,我改改,把c设置成缩进2,其余默认 |
Collaborator
Author
|
fixed |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
问题描述
N/A方案与实现