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
Add black-doc to pre-commit #4120
Conversation
Codecov Report
@@ Coverage Diff @@
## main #4120 +/- ##
=======================================
Coverage 95.57% 95.57%
=======================================
Files 95 95
Lines 20250 20282 +32
=======================================
+ Hits 19353 19384 +31
- Misses 897 898 +1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's hold back on this until any issues regarding #4119 are settled and git blame ignore revs items are added here.
Reminder to add:
to |
@adeak Could you approve this? I cannot wait using black-doc 🙂 |
I still have to add these. Will do today. |
Also #4120 (comment). |
Should be good to go @adeak. |
Co-authored-by: Andras Deak <adeak@users.noreply.github.com>
Redo of #4119 whereby only black doc is added. Style check is expected to fail. We will have a follow up PR implementing the style changes.