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

chore: use infra-checkers static analysis settings, bump go to 1.18 and bump dependencies #131

Merged
merged 1 commit into from Aug 23, 2022

Conversation

sigilioso
Copy link
Contributor

@sigilioso sigilioso commented Jul 28, 2022

No description provided.

@sigilioso sigilioso changed the title chore: use infra-checkers static analysis settings, bump go to 1.18 a… chore: use infra-checkers static analysis settings, bump go to 1.18 and bump dependencies Jul 28, 2022
@sigilioso sigilioso force-pushed the chore/static-analysis-settings branch from cb0f08c to ab4c618 Compare July 28, 2022 14:43
@sigilioso sigilioso force-pushed the chore/static-analysis-settings branch from ab4c618 to 27bdc9e Compare July 28, 2022 14:45
@sigilioso sigilioso requested a review from a team July 28, 2022 14:50
@sigilioso sigilioso marked this pull request as ready for review July 28, 2022 14:50
@kang-makes kang-makes marked this pull request as draft August 2, 2022 08:48
@sigilioso sigilioso marked this pull request as ready for review August 22, 2022 14:33
@@ -69,13 +69,18 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/setup-go@v2
with:
go-version: '1.18'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Needed to use golangci-lint-action v3. Review this for details.

Copy link
Member

@paologallinaharbur paologallinaharbur left a comment

Choose a reason for hiding this comment

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

thanks for taking care of this!

@@ -1,4 +1,4 @@
FROM alpine:3.15.4
FROM alpine:3.16.1
Copy link
Member

Choose a reason for hiding this comment

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

(not related to the PR)
Sometimes we pin the patch and sometimes we do not, we should at some point agree on one approach.

@sigilioso sigilioso merged commit 4e376f2 into main Aug 23, 2022
@sigilioso sigilioso deleted the chore/static-analysis-settings branch August 23, 2022 09:18
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.

None yet

2 participants