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

Move all the extra configurations to pyproject.toml #3478

Closed
Saransh-cpp opened this issue Oct 27, 2023 · 2 comments · Fixed by #3557
Closed

Move all the extra configurations to pyproject.toml #3478

Saransh-cpp opened this issue Oct 27, 2023 · 2 comments · Fixed by #3557
Labels
difficulty: easy A good issue for someone new. Can be done in a few hours priority: low No existing plans to resolve

Comments

@Saransh-cpp
Copy link
Member

Once #3301 is merged, we can move the configurations of pytest, ruff, and others to pyproject.toml
@Saransh-cpp Saransh-cpp added difficulty: easy A good issue for someone new. Can be done in a few hours priority: low No existing plans to resolve labels Oct 27, 2023
@agriyakhetarpal
Copy link
Member

I was going to open this issue, you beat me to it! 😄

@agriyakhetarpal
Copy link
Member

agriyakhetarpal commented Oct 27, 2023 •
edited

Some more tools whose configuration can be moved: cibuildwheel, coverage.py, and subsequently towncrier – after #3469 is merged)

@Rjchauhan18 Rjchauhan18 mentioned this issue Oct 30, 2023
Closed
@Saransh-cpp Saransh-cpp mentioned this issue Nov 25, 2023
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
difficulty: easy A good issue for someone new. Can be done in a few hours priority: low No existing plans to resolve
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Move ruff, coverage, and pytest configs to pyproject.toml Saransh-cpp/PyBaMM
2 participants
@Saransh-cpp @agriyakhetarpal