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.

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

gh-170: add scientific python recommended blacken-docs #250

Merged
merged 3 commits into from
Sep 25, 2024

Conversation

Saransh-cpp
Copy link
Member

Add blacken-docs for formatting of documentation files.

Refs: #170, #187

@Saransh-cpp Saransh-cpp added documentation Improvements or additions to documentation infrastructure Project infrastructure: dev tools, packaging, etc. labels Sep 19, 2024
@ntessore
Copy link
Collaborator

Has this been run over the repository with no changes?

@Saransh-cpp
Copy link
Member Author

Yes, everything is already in good shape.

@paddyroddy paddyroddy self-requested a review September 25, 2024 13:18
Copy link
Member

@paddyroddy paddyroddy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you 100% sure this is not already covered by ruff?

@paddyroddy
Copy link
Member

Are you 100% sure this is not already covered by ruff?

astral-sh/ruff#3792

@paddyroddy paddyroddy merged commit 2d6e21b into main Sep 25, 2024
10 checks passed
@paddyroddy paddyroddy deleted the saransh/blacken-docs branch September 25, 2024 14:44
Saransh-cpp added a commit that referenced this pull request Sep 25, 2024
Add blacken-docs for formatting of documentation files.

Refs: #170, #187

---------

Co-authored-by: Patrick J. Roddy <[email protected]>
@Saransh-cpp Saransh-cpp self-assigned this Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation infrastructure Project infrastructure: dev tools, packaging, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants