mc2 icon indicating copy to clipboard operation
mc2 copied to clipboard

Remove print statements before releasing on PyPi

Open jochasinga opened this issue 4 years ago • 2 comments

There is current a few places where there are still print statements. Can we switch to logger instead?

https://github.com/mc2-project/mc2/blob/bfbec919c0e10a48b3655a4cde5b5a5640f9ef58/python-package/tests/azure/delete_azure_resources.py#L6

jochasinga avatar Aug 02 '21 21:08 jochasinga

@chester-leung are these print statements save to remove and use logger instead?

jochasinga avatar Aug 10 '21 19:08 jochasinga

Yes, we can replace these print statements with log messages. Currently I don't believe there's a logger configured for the tests as a whole though.

chester-leung avatar Aug 10 '21 19:08 chester-leung