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.

Sign up for GitHub

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

Jump to bottom

[pre-commit.ci] pre-commit autoupdate #7063

Merged
merged 3 commits into from
Apr 4, 2023
Merged

[pre-commit.ci] pre-commit autoupdate #7063

merged 3 commits into from
Apr 4, 2023

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Apr 3, 2023

pre-commit-ci bot and others added 3 commits April 3, 2023 19:44
@pre-commit-ci
[pre-commit.ci] pre-commit autoupdate
5a7e2ad 
updates:
- [github.com/psf/black: 23.1.0 → 23.3.0](psf/black@23.1.0...23.3.0)
- [github.com/PyCQA/bandit: 1.7.4 → 1.7.5](PyCQA/bandit@1.7.4...1.7.5)
- [github.com/Lucas-C/pre-commit-hooks: v1.4.2 → v1.5.1](Lucas-C/pre-commit-hooks@v1.4.2...v1.5.1)
- [github.com/tox-dev/tox-ini-fmt: 0.6.1 → 1.0.0](tox-dev/tox-ini-fmt@0.6.1...1.0.0)
@hugovk
Don't remove tabs from hopper.gd
f46594f
@hugovk
Run tox-ini-fmt
6c057b3
@hugovk hugovk force-pushed the pre-commit-ci-update-config branch from 348cab3 to 6c057b3 Compare April 3, 2023 20:08
@hugovk
Copy link
Member

hugovk commented Apr 3, 2023

The updated remove-tabs initially modified hopper.gd:

Tabs remover.............................................................Failed
- hook id: remove-tabs
- exit code: 1
- files were modified by this hook

Substituting tabs in: Tests/images/hopper.gd by 4 whitespaces

https://results.pre-commit.ci/run/github/5171600/1680551062.qmBNy6zgRgOX4J7GvVrpOA

So I've added .gd files to its list of exclusions and updated this branch.

@radarhere radarhere merged commit 7d3a6aa into main Apr 4, 2023
@radarhere radarhere deleted the pre-commit-ci-update-config branch April 4, 2023 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hugovk hugovk hugovk approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hugovk @radarhere