-
-
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 for CRAN release #713
Conversation
Unit Tests Summary 1 files 53 suites 2m 39s ⏱️ For more details on these failures and errors, see this check. Results for commit 1bc0e77. ♻️ This comment has been updated with latest results. |
Code Coverage Summary
Diff against main
Results for commit: 7a4a00c Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Unit Test Performance Difference
Additional test case details
Results for commit 8740174 ♻️ This comment has been updated with latest results. |
Signed-off-by: b_falquet <[email protected]>
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.
Thank you, ready for submission
Signed-off-by: Liming <[email protected]>
I think we need to remove the "Remotes" field again. |
since everything is now open-source, do you need "remotes" field ever again? |
yes I think we can remove it forever |
joe's suggestion Co-authored-by: Joe Zhu <[email protected]> Signed-off-by: b_falquet <[email protected]>
Signed-off-by: Liming <[email protected]>
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.
minor comment
@BFalquet I reduce the size of our test data, leading to all the snapshot changes. (it can be further reduced, if needed) However this should not be critical (as no functionality is updated) |
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.
The tests for lbt14
and chevron-methods
were still a bit slow (~30sec), I trimmed the data in the initial part of the test to speed up the execution to ~15sec locally. rspt01
snapshot tests were failing but it could be a rounding error, I commit this change separately. Note, tests were performed with the CRAN version of formatters
, rlistings
, rtables
, tern
and dunlin
.
tern_0.9.3, rlistings_0.2.7, rtables_0.6.6, formatters_0.5.5
No description provided.