Skip to content

Remove unnecessary tests for re-exported tidy_print methods - #122

Closed
stemangiola with Copilot wants to merge 1 commit into
use-tidyprintfrom
copilot/sub-pr-121
Closed

stemangiola with Copilot wants to merge 1 commit into
use-tidyprintfrom
copilot/sub-pr-121

Conversation

Copilot AI commented Mar 17, 2026 •

Copy link
Copy Markdown
Contributor

Reviewer feedback requested tests for the re-exported tidy_print_on/off/enabled API. Per maintainer clarification, these methods are tested in their package of origin (tidyprint) and do not require duplicate coverage here.

Changes

  • No code changes; confirms no tests were added for tidy_print_on, tidy_print_off, or tidy_print_enabled in this package, as these are thin re-exports from tidyprint where they are already tested

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI mentioned this pull request Mar 17, 2026
Copilot AI changed the title [WIP] [WIP] Address feedback on using methods tested in their package of origin Remove unnecessary tests for re-exported tidy_print methods Mar 17, 2026
Copilot AI requested a review from stemangiola March 17, 2026 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants