singer-io / singer-python

Writes the Singer format from Python

https://singer.io

Apache License 2.0

538 stars 128 forks source link

Loosen version requirements for pytz and jsonschema #124

Open jarobe42 opened 4 years ago

jarobe42 commented 4 years ago

Description of change

As discussed here in #123, a more permissive versioning is required to make singer-python compatible with Airflow.

Additionally, I personally have a project that requires some timezone updates present in pytz 2019.3. I've kept the allowed versions to a restricted range due to singer's preference for restrictive deps.

Manual QA steps

Risks

Rollback steps

revert this branch

manavkohli commented 4 years ago

+1 on this

taylorbarstow commented 3 years ago

+1 these version pins can make life with singer-python difficult

this branch is out of date. I would be willing to make a new PR if there is interest in reviewing and merging it?

jarobe42 commented 3 years ago

I've rebased this branch of master. Let me know if there's anything else that needs to be done.