GitHub Action CI: Add Go source file format check#6090
Open
KobeArthurScofield wants to merge 6 commits intoXTLS:mainfrom
Open
GitHub Action CI: Add Go source file format check#6090KobeArthurScofield wants to merge 6 commits intoXTLS:mainfrom
KobeArthurScofield wants to merge 6 commits intoXTLS:mainfrom
Conversation
Member
|
1 还得先全core format一遍 不然自己炸自己 |
Contributor
Author
暂时来说为了测试效果有意为之，后面再看什么时机 format 一次（现在格式化问题确实一堆）
安排上了，不过 |
Contributor
Author
|
给 vformat 增加了格式检查功能（默认还是格式化并写出），暂时禁用 gci 避免在部分文件出现的格式化打架问题 gofumpt 有点小问题，但是不影响整体用到的功能（格式化和检查都没什么大问题），也没想到有什么解决方法。
|
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
2 participants
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.
#6057 (comment)
全局检查一遍，有问题直接列出文件路径以及具体更改并且错误码触发 Actions 错误，不写回仓库