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 and include known issue to trivyignore #52

Merged
merged 2 commits into from Aug 29, 2022

Conversation

sigilioso
Copy link
Contributor

@sigilioso sigilioso commented Jul 28, 2022

No description provided.

@CLAassistant
Copy link

CLAassistant commented Jul 28, 2022

CLA assistant check
All committers have signed the CLA.

@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 2 times, most recently from e8eac45 to 055624e Compare July 28, 2022 15:26
@sigilioso sigilioso changed the title chore: use infra-checkers static analysis settings, bump go to 1.18 and bump dependencies chore: use infra-checkers static analysis settings, bump go to 1.18 and bump alpine image version Jul 28, 2022
@sigilioso sigilioso marked this pull request as ready for review August 22, 2022 14:33
@sigilioso sigilioso force-pushed the chore/static-analysis-settings branch from 055624e to 1d50ff2 Compare August 23, 2022 07:43
@@ -19,13 +19,18 @@ jobs:
runs-on: ubuntu-20.04
steps:
- uses: actions/checkout@v2
- uses: actions/setup-go@v2
with:
go-version: ${{ env.GO_VERSION }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

it is needed when using golangci-lint-action v3, check this for details.

@sigilioso sigilioso requested a review from a team August 23, 2022 12:30
@paologallinaharbur
Copy link
Member

paologallinaharbur commented Aug 25, 2022

The title says bump alpine image version, but I do not see such change, did you forget to push it? (we could also merge one of the many already opened)

@sigilioso
Copy link
Contributor Author

The title says bump alpine image version, but I do not see such change, did you forget to push it? (we could also merge one of the many already opened)

I did not, but it was also updated in #53 , so after the rebase the change is not needed anymore, I'm updating the title to match the current changes.

@sigilioso sigilioso changed the title chore: use infra-checkers static analysis settings, bump go to 1.18 and bump alpine image version chore: use infra-checkers static analysis settings and include known issue to trivyignore Aug 25, 2022
@sigilioso sigilioso merged commit eaffa2e into master Aug 29, 2022
@sigilioso sigilioso deleted the chore/static-analysis-settings branch August 29, 2022 06:42
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

3 participants