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

ci: Add GitHub Actions for golangci-lint. #7345

Merged
merged 9 commits into from
Mar 18, 2021

Conversation

danielmai
Copy link
Contributor

@danielmai danielmai commented Jan 21, 2021

This adds golangci-lint to run for every PR.

https://golangci-lint.run/usage/install#github-actions


This change is Reviewable

@github-actions github-actions bot added the area/integrations Related to integrations with other projects. label Jan 21, 2021
I expect to see bot comments on the GitHub PR.
Revert before merging.
It's possible that the action times out because of the build cache. The default
timeout is 1-minute, so this change doubles the timeout duration.

https://github.com/dgraph-io/dgraph/pull/7345/checks?check_run_id=1884418647

run golangci-lint
  Running [/home/runner/golangci-lint-1.36.0-linux-amd64/golangci-lint run --out-format=github-actions --new-from-patch=/tmp/tmp-2680-QAynk1d0PEVf/pull.patch --new=false --new-from-rev=] in [] ...
  level=error msg="Timeout exceeded: try increasing it by passing --timeout option"
@danielmai danielmai merged commit 3caa276 into master Mar 18, 2021
@danielmai danielmai deleted the danielmai/github-actions-golangci-lint branch March 18, 2021 17:43
aman-bansal pushed a commit that referenced this pull request Apr 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/integrations Related to integrations with other projects.
Development

Successfully merging this pull request may close these issues.

1 participant