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

Add instructions to set up pre-commit #593

Merged
merged 2 commits into from
Jul 25, 2023
Merged

Add instructions to set up pre-commit #593

merged 2 commits into from
Jul 25, 2023

Conversation

unalmis
Copy link
Collaborator

@unalmis unalmis commented Jul 22, 2023 •
edited
Loading

Pre-commit automatically runs linting stuff so it's less of a hassle for developers. Recommends developers to

pip install pre-commit  # or conda install -c conda-forge pre-commit
cd DESC
pre-commit install

@unalmis unalmis marked this pull request as ready for review July 22, 2023 04:38
@unalmis unalmis added run_benchmarks Run timing benchmarks on this PR against current master branch and removed run_benchmarks Run timing benchmarks on this PR against current master branch labels Jul 24, 2023
@f0uriest f0uriest merged commit 0753bf7 into master Jul 25, 2023
@f0uriest f0uriest deleted the precommit branch July 25, 2023 15:35
@unalmis unalmis added the documentation Add documentation or better warnings etc. label Jul 22, 2024
@unalmis unalmis self-assigned this Aug 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dpanici dpanici dpanici approved these changes

@f0uriest f0uriest f0uriest approved these changes

Assignees

@unalmis unalmis

Labels
documentation Add documentation or better warnings etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@unalmis @f0uriest @dpanici