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

Python3 support #9

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Python3 support #9

wants to merge 5 commits into from

Conversation

dvdme
Copy link

@dvdme dvdme commented Dec 13, 2017

Python 3 support (and logging and verbosity function)

  • Python 3 is regarded as the future of the language so it should be prefered
  • Cleaner usage of requests: changed json.loads(u._content) to the more clean usage u.json()

Update .gitignore

  • With standarts from github gitignores repo
  • Add .vscode/
  • Move to root repo folder

Added proper logging to console and a file

  • Logger can be configured in the config file
  • Added an working example config in the example file

Added verbosity function

  • Added verbosity function using the debug level in logger

Python 2 still works

- With standarts from github gitignores repo
- Add .vscode/
- Move to root repo folder
- Python 3 is regarded as the future of the language
  so it _should_ be prefered
- Cleaner usage of requests: changed json.loads(u._content)
  to the more clean usage u.json()
 - Logger can be configured in the config file
 - Added an working example config in the example file
 - Added verbosity function using the debug
   level in logger
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