Skip to content

ci: Improve performance of go lint and formatting#4831

Draft
johnsaigle wants to merge 1 commit into
wormhole-foundation:mainfrom
johnsaigle:ci-improve-lint-performance
Draft

ci: Improve performance of go lint and formatting#4831
johnsaigle wants to merge 1 commit into
wormhole-foundation:mainfrom
johnsaigle:ci-improve-lint-performance

Conversation

@johnsaigle

@johnsaigle johnsaigle commented May 21, 2026

Copy link
Copy Markdown
Contributor
  • Use git ls-files to retrieve check-in Go files for linting. This should be faster than using find to traverse all the subdirs and look up OS information.
  • Allow parallel golangci-lint runs so node/ and sdk/ run at the same time

@johnsaigle johnsaigle added enhancement New feature or request ci labels May 21, 2026
@johnsaigle johnsaigle marked this pull request as ready for review May 21, 2026 18:42
djb15
djb15 previously approved these changes Jun 4, 2026
@johnsaigle johnsaigle force-pushed the ci-improve-lint-performance branch from 77e5814 to dcd1234 Compare June 16, 2026 14:16
@johnsaigle johnsaigle force-pushed the ci-improve-lint-performance branch from dcd1234 to 4920f7e Compare June 16, 2026 15:01
- use git ls-files instead of find
- Allow parallel golangci-lint runs so node/ and sdk/ run at the same
  time
@johnsaigle johnsaigle force-pushed the ci-improve-lint-performance branch from 4920f7e to ac7d5ae Compare June 16, 2026 18:33

@SEJeff SEJeff left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nice!

@johnsaigle johnsaigle marked this pull request as draft June 18, 2026 18:52
@johnsaigle

Copy link
Copy Markdown
Contributor Author

Moving to draft mode for the moment so that we can merge in #4841 first. I'll come back to this afterward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants