lamby / django-slack

Slack integration for Django, using the templating engine to generate messages
https://django-slack.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
241 stars 57 forks source link

Update api.py #91

Closed jnahorniak closed 4 years ago

jnahorniak commented 4 years ago

I received the following error when using the latest version of django-slack with python 3.5.3 on Debian (in a virtual environment):

File "######/lib/python3.5/site-packages/django_slack/api.py", line 17 **kwargs, ^ SyntaxError: invalid syntax

The error disappeared after I edited my local version of api.py to remove the comma after **kwargs

lamby commented 4 years ago

I'm a little hesitant to continue to support Python 3.5 given that it support for this version was ended in 2017-08-08. Is there a particular reason why you are on this version?

jnahorniak commented 4 years ago

Yikes! I wasn't intentionally using an older version of python (in fact, I just did an apt-get install python3 yesterday). I'll update my python version and will get back to you if the bug persists.

Thanks for the quick reply!

Jas.

On 1/14/20 2:22 AM, Chris Lamb wrote:

I'm a little hesitant to continue to support Python 3.5 given that it support for this version was ended in 2017-08-08. Is there a particular reason why you are on this version?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/lamby/django-slack/pull/91?email_source=notifications&email_token=ADGVWOXIRTKDSA64OOCAZOTQ5WG6TA5CNFSM4KGKWCG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEI4CPJI#issuecomment-574105509, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADGVWORWWOTZRXMDYQE5IA3Q5WG6TANCNFSM4KGKWCGQ.

-- Senior Faculty Research Assistant II College of Earth, Ocean, and Atmospheric Sciences Oregon State University Corvallis, Oregon, USA

lamby commented 4 years ago

It is almost certainly this so will close the report ; let me know and I can reopen it, of course. :)