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

Updated to support python3, added .gitignore and requirements.txt, made o365creeper.py executable #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

retrodaredevil
Copy link

@retrodaredevil retrodaredevil commented Aug 7, 2020

This PR adds parenthesis to the print() calls to support python3, but this script will still work in python2.

The added .gitignore was generated by https://www.gitignore.io, so it's basically guaranteed to cover most use cases.

I added a requirements.txt for easy installation.

I made o365creeper.py executable, which is useful on unix based systems so you can do ./o365creeper.py. I'm not really sure if that has an affect on Windows systems.

I made these changes a while ago meaning to make a PR then, but I didn't get around to it until now.

@gpi-atos
Copy link

LGTM
I just had to the same thing on my side. This PR would be great for ease of use.

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.

2 participants