Skip to content

chore: replace print() statements with proper logging#5763

Closed
abhicodes0324 wants to merge 1 commit intolearningequality:unstablefrom
abhicodes0324:chore/replace-print-with-logging
Closed

chore: replace print() statements with proper logging#5763
abhicodes0324 wants to merge 1 commit intolearningequality:unstablefrom
abhicodes0324:chore/replace-print-with-logging

Conversation

@abhicodes0324
Copy link

Summary

This PR replaces remaining print() calls in production Python code with proper logging and Django command output (self.stdout / self.stderr), so output is consistent with project conventions.

While doing this cleanup, I also removed plain-text password logging from user creation logs.

Scope is intentionally small and does not change business logic.

References

No linked issue; maintenance cleanup.

Reviewer guidance

  • Confirm this is output/logging cleanup only.
  • Spot-check command files now use Django output methods.
  • Verify no sensitive data is logged.
  • Repo search for production print() calls now returns no matches.

@learning-equality-bot
Copy link

👋 Thanks for contributing!

We will assign a reviewer within the next two weeks. In the meantime, please ensure that:

  • You ran pre-commit locally
  • All issue requirements are satisfied
  • The contribution is aligned with our Contributing guidelines. Pay extra attention to Using generative AI. Pull requests that don't follow the guidelines will be closed.

We'll be in touch! 😊

Signed-off-by: Abhiswant Chaudhary <abhicodes0324@gmail.com>
@abhicodes0324 abhicodes0324 force-pushed the chore/replace-print-with-logging branch from 71a145e to 80c5480 Compare March 17, 2026 00:33
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