Possible to update to use newest version of pyyaml dependency?

civisanalytics / civis-python

Civis API Python Client

BSD 3-Clause "New" or "Revised" License

34 stars 26 forks source link

Possible to update to use newest version of pyyaml dependency? #292

Closed mcoirad-gmmb closed 5 years ago

mcoirad-gmmb commented 5 years ago

I think the latest pyyaml release is 5.1 but the requirements.txt lists only 3.99. This causes incompatibility with the newest version of the 'googleads' module for example which depends on 5.1. Possible to update?

keithing commented 5 years ago

I opened #293 to update this. It may take some time for a release, but after this merges this will fix the issue if you pull from master.

mcoirad-gmmb commented 5 years ago

Thanks! I will pull from here once it merges.