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

Update ci.yml #340

Merged
merged 2 commits into from
May 15, 2022
Merged

Update ci.yml #340

merged 2 commits into from
May 15, 2022

Conversation

x448
Copy link
Contributor

@x448 x448 commented May 15, 2022

Description

Revoke default permissions in CI.
Grant minimum required permission for CI build and test.
Add check-latest: true for setup-go v3.

x448 added 2 commits May 14, 2022 22:24
Revoke default permissions.
Grant minimum required permission for CI job.
Add Go version to test names:
name: test ${{matrix.os}} go-${{ matrix.go-version }}
Copy link
Owner

@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 great! 👍 Thanks!

@fxamacker fxamacker merged commit 8988cfe into fxamacker:master May 15, 2022
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