singer-io / singer-python

Writes the Singer format from Python

https://singer.io

Apache License 2.0

538 stars 128 forks source link

Update jsonschema version and other dependencies #148

Open ericlebail opened 3 years ago

ericlebail commented 3 years ago

Description of change

Update dependencies version especialy jsonschema.

Manual QA steps

Risks

Rollback steps

revert this branch

edgarrmondragon commented 1 year ago

Bump to this. singer-python should not be pinning deps, being a library means it's expected to be installed alongside arbitrary packages with overlapping requirements. If anyone's interested in the subject, https://iscinumpy.dev/post/app-vs-library/ and the related series of posts are a great read.

keithhackbarth commented 1 year ago

@Vi6hal or @sgandhi1311 - Any chance of getting this merged if you look in the PRs there's 4 or 5 versions of the same thing - seems like its a major blocker to using this repo! Also, recognizing this is open source - happy to do a PR review session if you want to give me temp maintainer access or anything like that that would help!