Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2166 +/- ##
=======================================
Coverage 61.95% 61.95%
=======================================
Files 393 393
Lines 33207 33207
Branches 4139 4139
=======================================
Hits 20572 20572
Misses 12635 12635
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
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 "@codex address that feedback".
|
@codex review |
|
Codex Review: Didn't find any major issues. What shall we delve into next? ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Renames every auto-generated
all.hpplegacy meta header (except dart8/dartpy8) toAll.hppso the public headers match the PascalCase convention used throughout thedart/andpython/dartpy/trees. The CMake helper is updated to emitAll.hpp,dart/All.hppreplaces the lowercase umbrella, and every include/doc reference now points to the new casing.Validated with
pixi run lint.Before creating a pull request
pixi run test-allto lint, build, and test your changes