-
Notifications
You must be signed in to change notification settings - Fork 795
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
DOCS: Clarify steps in the contributing guidelines #2569
Conversation
CONTRIBUTING.md
Outdated
Note that Altair code uses the [black](https://black.readthedocs.io/) | ||
code formatter, which you can apply to your modifications by installing | ||
and running black on the local directory: | ||
and [flake8](https://flake8.pycqa.org/en/latest/) code formatters |
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.
nitpick: flake8 is a linter, not a code formatter
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.
Thank you, updated!
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.
Looks great, thanks for putting this together! Just one minor nitpick
Actions are failing because of #2575, not related to this PR |
#2575 is merged, can you rebase this one? |
01b89a4
to
7ab6590
Compare
Rebased and ready to merge! |
Thanks! |
Just noticed a few things in the contributing guideline as I was making my recent PRs.