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.

Sign up for GitHub

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

Jump to bottom

[linting] Add golangci-lint and GHA that runs it #36

Merged
merged 3 commits into from
Aug 29, 2022
Merged

[linting] Add golangci-lint and GHA that runs it #36

merged 3 commits into from
Aug 29, 2022

Conversation

mikeland73
Copy link
Contributor

Summary

  • Moves tests over to own workflow. Release jobs depend on it.
  • Adds golangci linter and runs it in test workflows.
  • Adds new pull request template.

How was it tested?

GHA

@mikeland73 mikeland73 merged commit 14c47ea into main Aug 29, 2022
@mikeland73 mikeland73 deleted the landau/add-linter branch August 29, 2022 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gcurtis gcurtis gcurtis approved these changes

@loreto loreto Awaiting requested review from loreto

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mikeland73 @gcurtis