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

Moving linting to GitHub Actions #276

Merged
merged 1 commit into from
Mar 2, 2023

Conversation

mattfarina
Copy link
Contributor

The GitHub Action can display, right in the code (i.e. PR), where the lint issue is at. This can provide more direct feedback to PR authors.

Note, dependabot will create PRs for the GitHub Action but not for the version of the linter that's specified in the configuration.

Copy link
Contributor

@KevinJoiner KevinJoiner left a comment

Choose a reason for hiding this comment

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

Looks alright to me not sure what we need to do to fix Fossa in CI.

The GitHub Action can display, right in the code (i.e. PR), where
the lint issue is at. This can provide more direct feedback to
PR authors.

Signed-off-by: Matt Farina <matt.farina@suse.com>
@cbron
Copy link
Contributor

cbron commented Mar 1, 2023

Not against CI, but want to confirm we can use github actions in all repos/orgs we own to be consistent.

@mattfarina
Copy link
Contributor Author

@cbron Having talked with both @macedogm and @jeff-hobbs, GitHub Actions should be fine to use for now.

Copy link
Contributor

@rmweir rmweir left a comment

Choose a reason for hiding this comment

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

LGTM

@mattfarina
Copy link
Contributor Author

@cbron I also spoke with EIO who is very good with GitHub Actions.

@mattfarina mattfarina merged commit 16ddd2e into rancher:master Mar 2, 2023
@mattfarina mattfarina deleted the gha-golanglint-ci branch March 2, 2023 21:43
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.

4 participants