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

Revamp local build and CI #8

Merged
merged 1 commit into from
Jul 2, 2024
Merged

Revamp local build and CI #8

merged 1 commit into from
Jul 2, 2024

Conversation

take-five
Copy link
Member

This commit adds some improvements to the local development experience and adds more checks to CI.

Namely:

  • Makefile with targets that make local development simpler: make build, make test, make audit, etc.
  • Same checks are added to CI to ensure code quality for each PR.
  • golangci-lint is added to the checks.
  • Linter warnings are fixed.
  • Tests are also run before the release.
  • Go updated to 1.22.

@take-five take-five force-pushed the add-ci-checks branch 2 times, most recently from 782d746 to f3aae22 Compare July 1, 2024 12:58
@take-five take-five marked this pull request as ready for review July 1, 2024 13:42
@take-five take-five requested a review from a team as a code owner July 1, 2024 13:42
@take-five take-five requested review from sliiser and removed request for a team July 1, 2024 13:42
This commit adds some improvements to the local development experience
and adds more checks to CI.

Namely:
- `Makefile` with targets that make local development simpler:
  `make build`, `make test`, `make audit`, etc.
- Same checks are added to CI to ensure code quality for each PR.
- [golangci-lint][1] is added to the checks.
- Linter warnings are fixed.
- Tests are also run before the release.
- Go updated to 1.22.

[1]: https://github.com/golangci/golangci-lint
@take-five take-five merged commit 609f773 into master Jul 2, 2024
5 checks passed
@take-five take-five deleted the add-ci-checks branch July 2, 2024 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants