-
Notifications
You must be signed in to change notification settings - Fork 135
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
ci: test against numpy v2 #695
Conversation
📝 WalkthroughWalkthroughThe changes in the pull request focus on the Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant System
User->>System: Request to install dependencies
System->>System: Check optional dependencies
alt Test dependencies
System->>System: Include "numpy<2"
end
alt Other dependencies
System->>System: Include amber, docs, benchmark as specified
end
System->>User: Dependencies installed
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
CodSpeed Performance ReportMerging #695 will not alter performanceComparing 🎉 Hooray!
|
|
Submitted to ParmEd/ParmEd#1364 |
This is fixed. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## devel #695 +/- ##
=======================================
Coverage 85.07% 85.07%
=======================================
Files 81 81
Lines 7478 7478
=======================================
Hits 6362 6362
Misses 1116 1116 ☔ View full report in Codecov by Sentry. |
Summary by CodeRabbit
test
dependency now clearly specifies"numpy<2"
without additional notes.