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

chore: Make make lint faster #8268

Merged
merged 4 commits into from
Mar 28, 2022
Merged

chore: Make make lint faster #8268

merged 4 commits into from
Mar 28, 2022

Conversation

alexec
Copy link
Contributor

@alexec alexec commented Mar 28, 2022

Signed-off-by: Alex Collins [email protected]

Fixes #TODO

@alexec alexec enabled auto-merge (squash) March 28, 2022 15:57
@@ -207,8 +207,6 @@ jobs:
fetch-depth: 0
- run: cp server/static/files.go.stub server/static/files.go
- uses: golangci/golangci-lint-action@v2
with:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

use latest version

Signed-off-by: Alex Collins <[email protected]>
Signed-off-by: Alex Collins <[email protected]>
Signed-off-by: Alex Collins <[email protected]>
Copy link
Member

@terrytangyuan terrytangyuan left a comment

Choose a reason for hiding this comment

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

What's the speedup so far?

@alexec
Copy link
Contributor Author

alexec commented Mar 28, 2022

Lint take 2m, but is more reliable.

@alexec alexec merged commit 0ac3320 into master Mar 28, 2022
@alexec alexec deleted the lint branch March 28, 2022 19:57
alexec added a commit that referenced this pull request May 3, 2022
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.

None yet

2 participants