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

Pre commit checks #219

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

edgester
Copy link
Contributor

Added a pre-commit configuration. When enabled, this runs the following checks on all new/changed files:

  • Remove trailing whitespace
  • Block large files not added through git-lfs
  • All new cheat sheets must have the YAML front-matter with an optional syntax or tags fields
  • Check all YAML files with yamllint

This can be added to a github action for continuous integration.
This automates some style checks that were previously done by hand.

Don't save pyenv version configuration
Ran 'pre-commit run --all' on the repository.
No content changes were made.
New files must have a YAML front-matter header that includes the
'syntax' or 'tags' fields.
Copy link
Contributor

@EmilyGraceSeville7cf EmilyGraceSeville7cf left a comment

Choose a reason for hiding this comment

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

No fixes found, approved.
Thanks for a contribution! 🚀

.pre-commit-config.yaml Outdated Show resolved Hide resolved
Co-authored-by: Emily Grace Seville <EmilySeville7cfg@gmail.com>
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.

2 participants