-
-
Notifications
You must be signed in to change notification settings - Fork 106
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
Update development setup documentation #3074
Conversation
$ mamba activate base | ||
$ conda-lock install --name pudl-dev --mamba --dev environments/conda-lock.yml | ||
$ mamba deactivate | ||
$ mamba activate pudl-dev |
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.
I removed these details entirely, and defaulted to the instructions which use make
. If folks want to see the detailed commands, they are listed in the Makefile
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.
i think that's great
* We also use `ruff to format our code <https://docs.astral.sh/ruff/formatter/>`__. It | ||
serves as a much faster drop-in replacement for longtime crowd favorite `black | ||
<https://black.readthedocs.io/en/stable/>`__ |
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.
Swapped in ruff
for black
in the setup instructions.
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## dev #3074 +/- ##
=====================================
Coverage 88.7% 88.7%
=====================================
Files 90 90
Lines 10994 10995 +1
=====================================
+ Hits 9758 9759 +1
Misses 1236 1236 ☔ View full report in Codecov by Sentry. |
PR Overview
black
autoformatting instructions withruff
git config
to get nice conda lockfile merges.PR Checklist
dev
).