Skip to content

Update build and testing #491

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

Merged
merged 1 commit into from
Mar 2, 2025
Merged

Update build and testing #491

merged 1 commit into from
Mar 2, 2025

Conversation

SuperQ
Copy link
Contributor

@SuperQ SuperQ commented Mar 2, 2025

  • Migrate golangci-lint to GitHub actions.
  • Update minimum Go version to 1.22.0.
  • Update Go test matrix to test latest Go versions.
  • Enable dependabot for GitHub actions.
  • Update and pin GitHub actions to SHA for supply chain security.
  • golangci-lint: Replace exportloopref with copyloopvar.
  • golangci-lint: Remove deprecated disabled linters.
  • Fixup various golangci-lint issues.
  • Mark a number of gosec linting issues with nolint. These are all int/uint overflow warnings. These may be intentional, but we need to review them.

@SuperQ SuperQ requested a review from TimRots March 2, 2025 14:30
* Migrate golangci-lint to GitHub actions.
* Update minimum Go version to 1.22.0.
* Update Go test matrix to test latest Go versions.
* Enable dependabot for GitHub actions.
* Update and pin GitHub actions to SHA for supply chain security.
* golangci-lint: Replace exportloopref with copyloopvar.
* golangci-lint: Remove deprecated disabled linters.
* Fixup various golangci-lint issues.
* Mark a number of `gosec` linting issues with nolint. These are all
  int/uint overflow warnings. These may be intentional, but we need
  to review them.

Signed-off-by: SuperQ <superq@gmail.com>
@SuperQ SuperQ force-pushed the superq/build_matrix branch from a6d1afd to ef81d4d Compare March 2, 2025 14:33
@TimRots TimRots mentioned this pull request Mar 2, 2025
Copy link
Member

@TimRots TimRots left a comment

Choose a reason for hiding this comment

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

Looks good to me. Created the issue from your todo item as #492

@SuperQ SuperQ merged commit 65eefd7 into master Mar 2, 2025
14 checks passed
@SuperQ SuperQ deleted the superq/build_matrix branch March 2, 2025 19:16
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.

2 participants