-
Notifications
You must be signed in to change notification settings - Fork 6
Add lint composite action
#369
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
base: main
Are you sure you want to change the base?
Conversation
Introduces a new 'config' input to specify the path to the pre-commit config file in the lint action. Updates documentation and workflow to support this option, and renames test config files for consistency.
Introduces a new 'comment-on-success' input to the lint GitHub Action, allowing a success comment to be posted even if there were no prior lint failures. Updates documentation and workflow to support this feature, which is useful for testing and visibility.
|
@kenodegard I like prek and also use it in some repos. autofix is one thing, what do we do in regards to hook version bumps? role out renovate with the pre-commit manager enabled? With an org wide shared config like https://github.com/anaconda/renovate-config/blob/main/default.json? Also requires some harmonising with existing dependabot configs to avoid duplicated PRs. |
|
Warning Working directory: ✅ Lint issues fixedAll lint issues have been resolved. See https://github.com/conda/actions/actions/runs/21216927394 for details. |
|
Warning Working directory:
|
Description
Introduces a new
lintaction that replacespre-commit.ciwith prek, a fast Rust-based pre-commit runner.Features
.pre-commit-config.yaml@conda-bot prek autofixto auto-commit fixes