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

Add CI infrastructure for pre-commit checks #61

Closed
wants to merge 3 commits into from

Conversation

jcfr
Copy link
Collaborator

@jcfr jcfr commented Aug 18, 2024

Introduce a noxfile with two sessions:

  1. pre-commit: Runs pre-commit checks on all files except the ones generated by cookiecutter.
  2. pre-commit-cookie: Runs pre-commit checks on a generated folder based on the cookiecutter template.

The approach was adapted from the implementation at https://github.com/scientific-python/cookie/, originally contributed by @henryiii.

Add corresponding GitHub Actions workflows in lint.yml to automate the pre-commit checks.

This commit updates the sources files based on the automatic fixes
applied using the "end-of-file-fixer" pre-commit hook introduced in
the subsequent commit.
This commit updates the sources files based on the automatic fixes
applied using the "mirrors-prettier" pre-commit hook introduced in
the subsequent commit.
Introduce a noxfile with two sessions:
1. pre-commit: Runs pre-commit checks on all files except the ones generated by cookiecutter.
2. pre-commit-cookie: Runs pre-commit checks on a generated folder based on the cookiecutter template.

The approach was adapted from the implementation at https://github.com/scientific-python/cookie/, originally contributed by @henryiii.

Add corresponding GitHub Actions workflows in lint.yml to automate the pre-commit checks.
@jcfr jcfr closed this Aug 18, 2024
@jcfr jcfr deleted the add-ci-infrastructure branch August 18, 2024 15:50
@jcfr
Copy link
Collaborator Author

jcfr commented Aug 18, 2024

Superseded by #62

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.

1 participant