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.

Sign up for GitHub

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

Jump to bottom

Use logger for printing info to console #53

Closed
alyssadai opened this issue Jan 9, 2024 · 1 comment · Fixed by #77
Closed

Use logger for printing info to console #53

alyssadai opened this issue Jan 9, 2024 · 1 comment · Fixed by #77
Assignees
@surchs
Labels
released This issue/pull request has been released.

Comments

@alyssadai
Copy link
Contributor

alyssadai commented Jan 9, 2024 •
edited by surchs
Loading

  • This will replace all existing warnings and print statements
  • Tests also need to be updated

Replace

  • print with logging.info
  • warning with logging.warning

Keep
@surchs surchs added this to Neurobagel Jan 9, 2024
@alyssadai alyssadai mentioned this issue Jan 11, 2024
Merged
6 tasks
@alyssadai alyssadai moved this to Backlog in Neurobagel Jan 11, 2024
@alyssadai alyssadai moved this from Backlog to Specify - Active in Neurobagel Jan 31, 2024
@surchs surchs moved this from Specify - Active to Specify - Done in Neurobagel Mar 21, 2024
@surchs surchs moved this from Specify - Done to Implement - Active in Neurobagel Mar 22, 2024
@surchs surchs self-assigned this Mar 22, 2024
@surchs surchs mentioned this issue Mar 25, 2024
Merged
7 tasks
@surchs surchs moved this from Implement - Active to Implement - Done in Neurobagel Mar 25, 2024
@alyssadai alyssadai moved this from Implement - Done to Review - Active in Neurobagel Mar 26, 2024
@github-project-automation github-project-automation bot moved this from Review - Active to Review - Done in Neurobagel Mar 27, 2024
@surchs
Copy link
Contributor

surchs commented Apr 11, 2024

🚀 Issue was released in v0.1.0 🚀

@surchs surchs added the released This issue/pull request has been released. label Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@surchs surchs

Labels
released This issue/pull request has been released.
Projects
Neurobagel
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[MNT] Switch to using logging for all non-user warnings and errors neurobagel/federation-api
2 participants
@surchs @alyssadai