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

Add shebang line, fix line terminators, file privileges, PEP8 #2

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

Conversation

pmandera
Copy link

Necessary changes to make it run directly from linux command line.
Because line terminators have been changed diff on the Punch.py file shows that all line were modified.

Suggest setting TODOTXT_CFG_FILE when todo.txt configuration file cannot be found.

Commit 05e2d3c fixes #3 (PEP8 compliance).

Paweł Mandera and others added 10 commits March 10, 2014 14:46
* add shebang line to allow execution directly from commandline
* fix line terminator to make it work on linux
* remove +x from COPYING
Suggest setting TODOTXT_CFG_FILE when can't find todo.txt configuration
file.
(checked with pep8 command line tool)

* removed whitespaces at blank lines
* split long lines
* normalized blank lines
* removed spaces before and after brackets
* spaces around commas and operators
* fixed indentation
* None/True/False matching with 'is' and 'is not'
* removed trailing spaces
* .has_key() replaced with 'in'
.gitignore now includes only Vim temporary files
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.

PEP8 compliance
1 participant