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 pretty-format-yaml pre-commit hook. #235

Merged
merged 2 commits into from
Apr 12, 2022
Merged

Conversation

euronion
Copy link
Collaborator

Change proposed in this Pull Request

Add a pre-commit hook for formatting yaml files.

Motivation and Context

We have a lot of yaml files and no automatic formatting. This fixes it.

How Has This Been Tested?

By running pre-commit run --all-files locally. This should not break anything, formatting only.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • I tested my contribution locally and it seems to work fine.
  • [n/a] I locally ran pytest inside the repository and no unexpected problems came up.
  • [n/a] I have adjusted the docstrings in the code appropriately.
  • [n/a] I have documented the effects of my code changes in the documentation doc/.
  • [n/a] I have added newly introduced dependencies to environment.yaml file.
  • I have added a note to release notes doc/release_notes.rst.
  • I have used pre-commit run --all to lint/format/check my contribution

@euronion euronion merged commit a19a1be into master Apr 12, 2022
@euronion euronion deleted the pre-commit/yamlformat branch April 12, 2022 15:00
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