-
Notifications
You must be signed in to change notification settings - Fork 0
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
maint: 2025 R1 update #630
Conversation
@@ -120,7 +120,7 @@ | |||
<dbKey xmlns="http://www.grantadesign.com/12/05/GrantaBaseTypes">MI_Restricted_Substances</dbKey> | |||
<recordGUID xmlns="http://www.grantadesign.com/12/05/GrantaBaseTypes">92f90382-610a-4737-a18f-b153ea88e4ad</recordGUID> | |||
</MIProcessReference> | |||
<Identity>Primary processing, Metal extrusion, hot</Identity> | |||
<Identity>Metal extrusion, hot</Identity> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change isn't technically necessary. It was believed that the identity would override the name in the summary output, but the name in the database is always used instead.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #630 +/- ##
=======================================
Coverage 97.63% 97.63%
=======================================
Files 18 18
Lines 2542 2542
=======================================
Hits 2482 2482
Misses 60 60
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes in the scripts look reasonable and as expected. I'll take your word on the changes to the test results!
Update tests and database preparation scripts for 2025 R1 release.
Test fixes required by changes to data in 2025 R1 database.