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 fixes #2669

Merged
merged 24 commits into from
Oct 31, 2023
Merged

pre-commit fixes #2669

merged 24 commits into from
Oct 31, 2023

Conversation

pat-s
Copy link
Contributor

@pat-s pat-s commented Oct 28, 2023

Following #2650

Via pre-commit run --all-files

@anbraten Could you enable https://pre-commit.ci/ for the org?

@anbraten
Copy link
Member

Could you enable https://pre-commit.ci/ for the org?

@pat-s

@woodpecker-bot
Copy link
Collaborator

woodpecker-bot commented Oct 29, 2023

Deployment of preview was successful: https://woodpecker-ci-woodpecker-pr-2669.surge.sh

@pat-s
Copy link
Contributor Author

pat-s commented Oct 29, 2023

Thanks! Good to go from my side.

@pat-s
Copy link
Contributor Author

pat-s commented Oct 29, 2023

@anbraten One last thing would be to "require" the precommit check to pass in the repo setting

@anbraten
Copy link
Member

@anbraten One last thing would be to "require" the precommit check to pass in the repo setting

It's required that all checks have to pass anyway, so this should already work.

docs/docs/92-development/08-swagger.md Outdated Show resolved Hide resolved
docs/docs/91-migrations.md Outdated Show resolved Hide resolved
.pre-commit-config.yaml Outdated Show resolved Hide resolved
@anbraten anbraten merged commit 856b2ef into main Oct 31, 2023
5 of 6 checks passed
@anbraten anbraten deleted the precommit-fixes branch October 31, 2023 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants