Migrate from legacy setup.py to modern pyproject.toml#2043
Migrate from legacy setup.py to modern pyproject.toml#2043jslee02 merged 1 commit intodartsim:mainfrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2043 +/- ##
=======================================
Coverage 58.90% 58.90%
=======================================
Files 385 385
Lines 34578 34578
Branches 3883 3883
=======================================
Hits 20367 20367
Misses 14211 14211
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting
jeongseok-meta
force-pushed
the
pyproject_migration
branch
from
9efa4ad to
c84fdcf
Compare
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
jeongseok-meta
force-pushed
the
pyproject_migration
branch
2 times, most recently
from
4885a6b to
d99b7e4
Compare
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
jeongseok-meta
force-pushed
the
pyproject_migration
branch
from
d99b7e4 to
3c2bf99
Compare
|
@codex review |
|
Codex Review: Didn't find any major issues. What shall we delve into next? ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
2 participants
Removes
setup.pyentirely and migrates to a purepyproject.tomlbuild configuration using scikit-build-core as the build backend.Before creating a pull request
pixi run lintand verify withpixi run check-lint-DDART_TREAT_WARNINGS_AS_ERRORS=ONand resolve warningsBefore merging
CHANGELOG.md