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 code to work with Python 3.6 #21

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

Conversation

Mestalbet
Copy link

The file had the wrong name and wouldn't run due to "raw_input" (now called input) and "unicode" (now called str) functions. Also, expected to receive u'' which is no longer necessary in Python 3.

@fabriceb
Copy link
Owner

Thanks @Mestalbet for this! I am happy to merge, I am just surprised that the diff is 100% of the file content, why is that?

@Mestalbet
Copy link
Author

Actually, I am not sure why it's 100% diff. That's super weird. It probably has to do with a combination of (a) me not remembering anymore and (b) me being a bit of an idiot somehow. :)

@Mestalbet
Copy link
Author

Ah - it's because the comments give the usage as "python gcalcron2.py" but the script is called "gcaclron.py" so I changed the file name to gcalcron2.py to conform to the usage instructions. Since there is no "gcalcron2.py" in your repo it's a 100% diff.

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