Skip to content

Use logging instead of print in RewritePass#2835

Merged
justinchuby merged 1 commit intomicrosoft:mainfrom
tgolsson:ts/spammy-print
Mar 11, 2026
Merged

Use logging instead of print in RewritePass#2835
justinchuby merged 1 commit intomicrosoft:mainfrom
tgolsson:ts/spammy-print

Conversation

@tgolsson
Copy link
Contributor

@tgolsson tgolsson commented Mar 3, 2026

Makes it possible to filter this out using the regular logging mechanics in Python.

@justinchuby justinchuby enabled auto-merge (squash) March 11, 2026 14:56
@justinchuby justinchuby changed the title use logging instead of print in RewritePass Use logging instead of print in RewritePass Mar 11, 2026
@justinchuby justinchuby disabled auto-merge March 11, 2026 14:56
@justinchuby justinchuby enabled auto-merge (squash) March 11, 2026 14:56
@codecov
Copy link

codecov bot commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.80%. Comparing base (3f197a2) to head (f43f6ff).
⚠️ Report is 9 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2835   +/-   ##
=======================================
  Coverage   71.79%   71.80%           
=======================================
  Files         239      239           
  Lines       29018    29020    +2     
  Branches     2864     2864           
=======================================
+ Hits        20833    20837    +4     
+ Misses       7213     7212    -1     
+ Partials      972      971    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@justinchuby justinchuby merged commit 4c4f7a0 into microsoft:main Mar 11, 2026
27 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

2 participants