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 #1684

Merged
merged 3 commits into from
Feb 24, 2021
Merged

Pre-commit #1684

merged 3 commits into from
Feb 24, 2021

Conversation

ivirshup
Copy link
Member

Initial PR for pre-commit.

  • Only uses black for checks.
  • Adds CI task
  • Adds dev docs

@ivirshup
Copy link
Member Author

turns out you have to add the github action yml in the first commit for it to register.

@ivirshup
Setup pre-commit
99b1d03 
* Add github action to run pre-commit
* Remove black from azure pipelines
* Remove blackdiff test case
* Document pre-commit
@ivirshup
Add pre-commit, starting with black
d040ce9
@ivirshup
Fix some comment labelling in palettes
d654852
@ivirshup ivirshup merged commit 5db49c2 into scverse:master Feb 24, 2021
This was referenced Feb 24, 2021
Merged
Closed
@ivirshup ivirshup mentioned this pull request Apr 3, 2021
Merged
6 tasks
@Zethson Zethson mentioned this pull request May 21, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ivirshup