GitHub Action CI: Add Go source file format check#6090
Merged
Conversation
Member
|
1 还得先全core format一遍 不然自己炸自己 |
Contributor
Author
暂时来说为了测试效果有意为之，后面再看什么时机 format 一次（现在格式化问题确实一堆）
安排上了，不过 |
Contributor
Author
|
给 vformat 增加了格式检查功能（默认还是格式化并写出），暂时禁用 gci 避免在部分文件出现的格式化打架问题 gofumpt 有点小问题，但是不影响整体用到的功能（格式化和检查都没什么大问题），也没想到有什么解决方法。
|
Contributor
Author
Member
|
@KobeArthurScofield rebase main 然后修一下 fmt，这个 PR 还是得尽早合入主线，合了后我让其它 PR rebase 一下 |
Member
|
@KobeArthurScofield |
Contributor
Author
|
好了 |
Contributor
Author
|
OK good to go |
nebulabox
added a commit
to nebulabox/Xray-core
that referenced
this pull request
Jun 2, 2026
* commit '94ffd50060f1cfd5d7482ec90a23a92bdefdff68': (107 commits) Xray-core v26.6.1 Burst observatory: Fix init check (XTLS#6221) DNS outbound: Replace "reject" with "return" (`rCode` is 0 by default) (XTLS#6214) uTLS: Update `ModernFingerprints` map and `OtherFingerprints` map (XTLS#6181) Realm finalmask: Fix client punch peers (XTLS#6213) GitHub Action CI: Add Go source file format check (XTLS#6090) Burst observatory: Fix time compare, cancel pending ping on instance close or new schedule started (XTLS#6106) Finalmask: Add mkcp-legacy (UDP) to replace mkcp-* and legacy header-* (XTLS#6201) Sudoku finalmask: Harden UDP ReadFrom() against invalid packets (XTLS#6185) XICMP finalmask: Refactor & Speed up; Add multi `ips` and `dgram` mode (client) (XTLS#6168) Salamander finalmask: Support `packetSize` (Gecko in Hysteria v2.9.2) (XTLS#6198) Finalmask: Add Realm (UDP hole punching in Hysteria v2.9.1) (XTLS#6137) README.md: Add flutter_vless to Xray Wrapper in Others (XTLS#6197) DNS: Avoid panic on domain too long (XTLS#6207) header-custom finalmask: Remove headerConnMode headerReadAddrAware interface (XTLS#6193) noise finalmask: Better `reset` (XTLS#6188) DNS: Avoid passing domain to WriteTo func (XTLS#6163) Bump golang.org/x/net from 0.54.0 to 0.55.0 (XTLS#6192) Socks5 server: More standard UDP ASSOCIATE (RFC 1928) (XTLS#6149) Hysteria server: `tls.WithNextProto("h3")` by default (XTLS#6186) ... # Conflicts: # core/core.go
Collaborator
Contributor
Author
|
|
Member
|
因为AI的缘故github的活跃仓库数量涨了几倍有余 而AI又很擅长编写大量CI加上它们commit又多加之action的免费额度给的很慷概导致runner不太稳定 甚至组服务器用的RAM也一直在涨 |
Collaborator
|
感觉 gh 快撑不住了早晚要缩水免费额度 |
RPRX
pushed a commit
that referenced
this pull request
Jun 2, 2026
…e repository (#6222) #6221 (comment) Completes #6090 (comment)
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
4 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 错误，不写回仓库