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: add config and small fixes #390

Merged
merged 7 commits into from
Apr 11, 2023

Conversation

consideRatio
Copy link
Member

@consideRatio consideRatio commented Apr 8, 2023

The JupyterHub github organization has adopted pre-commit and the automation to run pre-commit via pre-commit.ci quite systematically. I've waited to make this PR until getting a few recently active PRs merged, which they now are.

Note that all autoformatting is put in the final commit by pre-commit.ci's automation, and all manual changes in previous commits by me.

@consideRatio
Copy link
Member Author

Rebased to handle merge conflicts.

@yuvipanda yuvipanda merged commit 3dde433 into jupyterhub:main Apr 11, 2023
@yuvipanda
Copy link
Contributor

Thanks @consideRatio

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.

Consider using black/pre-commit for nicer code
2 participants