-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
[skip vbump] prepare release #516
Conversation
🧪 Code Coverage Summary
Diff against main
Results for commit: 5eb26c0aa07732dcc060d84904aa71836cc32fb9 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
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.
Looks good, thank you. For the note relative to the size of the package, I wonder if this is ok or if we should trim syn_data
@@ -1,7 +1,7 @@ | |||
Type: Package | |||
Package: chevron | |||
Title: Standard TLGs for Clinical Trials Reporting | |||
Version: 0.1.5 | |||
Version: 0.2.0 |
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.
Directly version 0.2 ?
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.
it should be. we are introducing many breaking changes in chevron so it is not good to continue with 0.1.x
trimming of data can happen in next increment |
autovalidator passes. we can add back the remotes field and re-tag |
fix check notes and update snapshots