Skip to content

Conversation

@ANJAN672
Copy link

Description

Replace print statements with proper logging in repo_load_controller.py

Fixes #3566

Changes

  • Replace 15 print() calls with appropriate logger methods
  • Uses logger.debug(), logger.info(), logger.warning(), and logger.error() based on context

Testing

  • Verified syntax with python3 -m py_compile
  • Logger is already defined in the file (line 18)

@guptapratykshh
Copy link
Contributor

guptapratykshh commented Jan 12, 2026

Ensure you have a local copy of your branch by checking out the pull request locally via command line.
In your local branch, run: git rebase HEAD~2 --signoff
Force push your changes to overwrite the branch: git push --force-with-lease origin fix/replace-print-with-logging-repo-load-controller

@ANJAN672 ANJAN672 force-pushed the fix/replace-print-with-logging-repo-load-controller branch from 0a43e10 to 6ba24d6 Compare January 12, 2026 10:40
@ANJAN672 ANJAN672 requested a review from sgoggins as a code owner January 12, 2026 10:40
@guptapratykshh
Copy link
Contributor

i think you have squashed all the commits and then all the previous commits have been pushed ,please fix this

Signed-off-by: ANJAN672 <rockyanjan672@gmail.com>
@ANJAN672 ANJAN672 force-pushed the fix/replace-print-with-logging-repo-load-controller branch from 89ae559 to d25270a Compare January 12, 2026 10:49
@ANJAN672
Copy link
Author

fixed, rebased properly now. only my commit on top of main.

Copy link
Collaborator

@shlokgilda shlokgilda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

Replace print statements with logging in repo_load_controller.py

3 participants