Skip to content
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

Moved relevant decoration expansion into formatters for rlistings fix #813

Conversation

Melkiades
Copy link
Contributor

@Melkiades Melkiades commented Jan 9, 2024

Copy link
Contributor

github-actions bot commented Jan 9, 2024

Unit Tests Summary

    1 files     24 suites   1m 37s ⏱️
  198 tests   197 ✅ 0 💤 0 ❌ 1 🔥
1 509 runs  1 507 ✅ 0 💤 1 ❌ 1 🔥

For more details on these errors, see this check.

Results for commit 94fc60a.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Jan 9, 2024

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
Tabulation framework 💔 $19.29$ $+1.15$ $0$ $0$ $0$ $0$

Results for commit 5f1809b

♻️ This comment has been updated with latest results.

@Melkiades Melkiades closed this Jan 9, 2024
@Melkiades Melkiades deleted the 243_fix_nl_expansion@remove_decoration_nl_matrix_form@main branch January 9, 2024 18:15
Melkiades added a commit to insightsengineering/formatters that referenced this pull request Jan 19, 2024
…246)

Fixes the following issues

- [x] #243
- [x] #232
- [x] insightsengineering/rlistings#183
- [x] True fix for
insightsengineering/rlistings#155
- [x] insightsengineering/scda.test#99

To merge with insightsengineering/rlistings#192
(tests and misc) and
insightsengineering/rtables#813 (tests and
removed titles/footers newline handling)

---------

Signed-off-by: Davide Garolini <[email protected]>
Co-authored-by: 27856297+dependabot-preview[bot]@users.noreply.github.com <27856297+dependabot-preview[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant