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

Bump safer-golangci-lint.yml to 1.52.2 #42

Merged
merged 3 commits into from
May 15, 2023
Merged

Bump safer-golangci-lint.yml to 1.52.2 #42

merged 3 commits into from
May 15, 2023

Conversation

x448
Copy link
Owner

@x448 x448 commented May 14, 2023

Changes:

  • Bump Go to 1.20.
  • Bump actions/setup-go to v4.
  • Bump golangci-lint to 1.52.2.
  • Use defaults for pull_request.

@x448
Bump safer-golangci-lint.yml to 1.52.2
eb8560e 
Bump Go to 1.20
Bump actions/setup-go to v4
Bump golangci-lint to 1.52.2
@x448 x448 requested a review from fxamacker May 14, 2023 18:13
@x448
Copy link
Owner Author

x448 commented May 14, 2023

@fxamacker No rush, need review / rubber stamp (avoid hit to scorecard).

x448 added 2 commits May 14, 2023 19:59
@x448
Update safer-golangci-lint.yml
4fb44a1
@x448
Update cover.yml
b9f9ae7 
Sync with fxamacker/cbor and use Go 1.19.
fxamacker
fxamacker approved these changes May 15, 2023
View reviewed changes
Copy link
Collaborator

@fxamacker fxamacker 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!

Nit: maybe try go-version: '1.20' (version in quotes) in cover.yml.

@x448 x448 merged commit 05ce95b into master May 15, 2023
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fxamacker fxamacker fxamacker approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@x448 @fxamacker