Add lifecycle deprecation badges/warnings by edelarua · Pull Request #867 · insightsengineering/rtables (original) (raw)
CLA Assistant Lite bot ✅ All contributors have signed the CLA
I have read the CLA Document and I hereby sign the CLA
Unit Tests Summary
1 files 25 suites 1m 24s ⏱️
204 tests 202 ✅ 0 💤 2 ❌
1 527 runs 1 525 ✅ 0 💤 2 ❌
For more details on these failures, see this check.
Results for commit 4fe287a.
♻️ This comment has been updated with latest results.
Unit Test Performance Difference
| Test Suite | StatusStatusStatus | Time on main | ±Time±Time±Time | ±Tests±Tests±Tests | ±Skipped±Skipped±Skipped | ±Failures±Failures±Failures | ±Errors±Errors±Errors |
|---|---|---|---|---|---|---|---|
| Exporters | 💚 | 18.5118.5118.51 | −1.88-1.88−1.88 | 000 | 000 | 000 | 000 |
| Pagination | 💚 | 15.1515.1515.15 | −1.41-1.41−1.41 | 000 | 000 | 000 | 000 |
| Tabulation framework | 💚 | 19.6319.6319.63 | −1.62-1.62−1.62 | 000 | 000 | +1+1+1 | 000 |
Additional test case details
| Test Suite | StatusStatusStatus | Time on main | ±Time±Time±Time | Test Case |
|---|---|---|---|---|
| Tabulation framework | 💚 | 10.4510.4510.45 | −1.13-1.13−1.13 | qtable_works |
Results for commit 481fb1e
♻️ This comment has been updated with latest results.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see you fixed only the deprecated functions. I think we do not need to have the stable badge for now. What you think?
Lgtm anyway ;) Thanks Emily!
I see you fixed only the deprecated functions. I think we do not need to have the stable badge for now. What you think?
I think the stable badge can be added separately, since it only affects documentation.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
[ Show hidden characters]({{ revealButtonHref }})