Python updates #97

Merged
merged 6 commits into from Mar 1, 2012

2 participants

@rickeyski

Hi,

I made a run through and made two additions as well as several bugfixes that I found.

  1. I added js-beautify to the scripts list in setup.py so it is properly created in /usr/bin when a user runs setup.py install.

  2. I added exception handling if the beautify_file function is passed a file that does not exist or cannot be opened, this also ensures the output file will be properly closed.

The stylistic fixes are, removed 5 trailing semi-colons, fixed a method body where the indent was off by one space, changed the --h option to -h, and changed the getopt error handling to match the rest of the main function.

All the best,
~Rickey

@esseks esseks merged commit 6162db3 into beautify-web:master Mar 1, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment