Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix pre-commit mangling test data #273

Merged
merged 2 commits into from
Nov 21, 2023
Merged

Conversation

jfrost-mo
Copy link
Member

@jfrost-mo jfrost-mo commented Nov 21, 2023

This allows adding test data much more easily.
Fixes #254

@jfrost-mo jfrost-mo added the bug Something isn't working label Nov 21, 2023
@jfrost-mo jfrost-mo self-assigned this Nov 21, 2023
@jfrost-mo jfrost-mo added the cleanup Non-functional improvement label Nov 21, 2023
Copy link
Contributor

github-actions bot commented Nov 21, 2023

Coverage

@jfrost-mo jfrost-mo changed the title Increase pre-commit file size limit to 10 MiB Fix pre-commit mangling test data Nov 21, 2023
@jfrost-mo jfrost-mo marked this pull request as ready for review November 21, 2023 11:24
@jfrost-mo jfrost-mo added small Small issue that should be quick to resolve and removed cleanup Non-functional improvement labels Nov 21, 2023
@Sylviabohnenstengel Sylviabohnenstengel merged commit 22c814d into main Nov 21, 2023
5 checks passed
@Sylviabohnenstengel Sylviabohnenstengel deleted the 254_pre-commit_tweaks branch November 21, 2023 16:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working small Small issue that should be quick to resolve
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make pre-commit checks more sane
2 participants