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

Ensure print_callback runs before formatting print args #49

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bscriver123
Copy link

Description

This pull request addresses the issue where the print_callback needs to be executed before formatting print arguments. Currently, the issue arises as repr() generates output that is not valid code, causing a break in the black formatting of the output.

Changes Made

  • Added a pre-execution step for print_callback to ensure it runs before formatting print arguments. This helps prevent repr() from creating output that breaks the black formatting of the output.

How to Test

  1. Ensure the print_callback is triggered before formatting print arguments.
  2. Verify that the black formatting of the output is not disrupted by invalid code generated by repr().

Related Issue

Link to the original issue

Additional Notes

Any additional information or context about the changes made can be added here.

This is a necessary adjustment to maintain proper formatting integrity and prevent issues caused by invalid code generated during execution.

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.

1 participant