Added argparse/optparse for cmdline options to pyjscompressor.py #767

Merged
merged 1 commit into from Aug 2, 2012

Conversation

Projects
None yet
2 participants
@happyalu
Contributor

happyalu commented Jul 30, 2012

No description provided.

+ compress_all(directory)
+except KeyboardInterrupt:
+ print('')
+ print('Compression Aborted')

This comment has been minimized.

Show comment Hide comment
@anthonyrisinger

anthonyrisinger Aug 2, 2012

Member

this try block needs to be indented and inside of if __name__ == __main__

@anthonyrisinger

anthonyrisinger Aug 2, 2012

Member

this try block needs to be indented and inside of if __name__ == __main__

@anthonyrisinger

This comment has been minimized.

Show comment Hide comment
@anthonyrisinger

anthonyrisinger Aug 2, 2012

Member

@happyalu, this all looks good, but see my comment -- the last try block is not inside if __name__ == __main__: check.

... once that's fixed should be no issue to pull.

Member

anthonyrisinger commented Aug 2, 2012

@happyalu, this all looks good, but see my comment -- the last try block is not inside if __name__ == __main__: check.

... once that's fixed should be no issue to pull.

@anthonyrisinger

This comment has been minimized.

Show comment Hide comment
@anthonyrisinger

anthonyrisinger Aug 2, 2012

Member

actually i'll just fix it :-) thanks @happyalu!

Member

anthonyrisinger commented Aug 2, 2012

actually i'll just fix it :-) thanks @happyalu!

anthonyrisinger added a commit that referenced this pull request Aug 2, 2012

Merge pull request #767 from happyalu/feature/pyjscompressor_multipro…
…cessing_cmdlineoptions

Added argparse/optparse for cmdline options to pyjscompressor.py

@anthonyrisinger anthonyrisinger merged commit 74e8190 into pyjs:master Aug 2, 2012

anthonyrisinger pushed a commit that referenced this pull request Aug 2, 2012

@happyalu

This comment has been minimized.

Show comment Hide comment
@happyalu

happyalu Aug 2, 2012

Contributor

@xtfxme Thanks a lot!

Contributor

happyalu commented Aug 2, 2012

@xtfxme Thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment