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

Change all instances of print to use logging for output #41

Open
1 task
nickviola opened this issue Jul 30, 2021 · 0 comments
Open
1 task

Change all instances of print to use logging for output #41

nickviola opened this issue Jul 30, 2021 · 0 comments
Labels
improvement This issue or pull request will add or improve functionality, maintainability, or ease of use

Comments

@nickviola
Copy link
Contributor

💡 Summary

We are currently using print and logging for output in various places in the project. We should standardize this and use logging across the project for output consistency.

Motivation and context

There is currently variation in how outputs are handled in the project so this will create consistency and improve maintainability.

Implementation notes

Replace calls to print() with appropriate logging() usage.

Acceptance criteria

Logging is used in place of print across the entire project.

  • print calls are replaced with logging usage
@nickviola nickviola added the improvement This issue or pull request will add or improve functionality, maintainability, or ease of use label Jul 30, 2021
@dav3r dav3r mentioned this issue Aug 3, 2021
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement This issue or pull request will add or improve functionality, maintainability, or ease of use
Projects
None yet
Development

No branches or pull requests

1 participant