install hirefire raises syntax error

ryanhiebert / hirefire

A Python lib to integrate with the HireFire service -- The Heroku Proccess Manager.

https://hirefire.readthedocs.io/

Other

34 stars 21 forks source link

install hirefire raises syntax error #24

Closed t-io closed 7 years ago

t-io commented 7 years ago

hirefire can be installed successfully but raises a syntax error When installing 0.0.4 with pip I get: File "/tmp/pip_build_root/HireFire/hirefire/procs/hotqueue.py", line 59 SyntaxError: invalid syntax -> def quantity(self. **kwargs): This typo is also in the latest master

ryanhiebert commented 7 years ago

Indeed. I guess that means nobody (except you now) is using hotqueue. Thanks for the bug report!

You can either 1) wait for someone to fix this bug (perhaps me later this evening), or 2) create a pull request with the fix. Once that's done, I can't give you a timeframe on when a release would be cut (it's not up to me), but you can install the updated branch directly from git with pip.

t-io commented 7 years ago

It's not critical for me, I don't use hotqueue in my project. I just stumble across this ugly syntax error installing hirefire via pip. 👉 PR #25