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

Add ruff to base test environment #2909

Merged
merged 1 commit into from
Nov 27, 2023
Merged

Add ruff to base test environment #2909

merged 1 commit into from
Nov 27, 2023

Conversation

francesco-ballarin
Copy link
Member

In preparation of a PR similar to FEniCS/basix#748 for dolfinx.

@francesco-ballarin
Copy link
Member Author

I can do the same for all remaining docker files too, if the transition in FEniCS/basix#748 is successful. I just wanted to open this before transition to ruff takes place in dolfinx as a reminder to avoid that the ruff PR plasters workflow files with pip installs.

@garth-wells garth-wells added docker ci Continuous Integration labels Nov 27, 2023
@garth-wells garth-wells added this pull request to the merge queue Nov 27, 2023
Merged via the queue into main with commit 095ef96 Nov 27, 2023
29 checks passed
@garth-wells garth-wells deleted the francesco/base-ruff branch November 27, 2023 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@garth-wells garth-wells garth-wells approved these changes

Assignees
No one assigned
Labels
ci Continuous Integration docker
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@francesco-ballarin @garth-wells