Remove print statements

Mester / demo-day-vikings

The Unlicense

2 stars 1 forks source link

Remove print statements #49

Closed anubhavcodes closed 8 years ago

anubhavcodes commented 8 years ago

This creates a problem with people using python3. Assigning myself.

anubhavcodes commented 8 years ago

@jasonmeeks @bdauer @Mester @tylerphillips55 there are a lot of print statements in master now, I am going to use the logging module to log the output on stdout. we can use our config.py to set the loglevel? Make sense?

anubhavcodes commented 8 years ago

The print statements were not removed completely, as I can see it in master, but instead they were fixed. Reopening and referencing it with a PR that removes all the print statements and replaces them with logging.

anubhavcodes commented 8 years ago

The last few commits has been because of the merges, it's still a pain in the ass. I have added logging, it now acts exactly as print statements, however with just one line of change we can redirect it to a file log somewhere. Take a look at it and let's merge it.