Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade dependencies #1814

Merged
merged 1 commit into from
Oct 8, 2019
Merged

Conversation

beckjake
Copy link
Contributor

@beckjake beckjake commented Oct 8, 2019

Fixes #1697
Fixes #1577

This does not address psycopg2, which we should deal with sometime but is a lot harder than these.

I've verified that werkzeug==0.15.6 and the current git master both work ok, so we should feel good about raising the werkzeug version to >=0.15.6,<2 whenever they release 1.0... unless they make more breaking changes before release. Supporting all those versions required only a one-line change to our imports, because we don't use any of the more advanced features, but it did drop support for 0.14.x.

Both colorama and sqlparse saw significant version changes but appear harmless, we don't really make use of everything they offer - I've preserved the old fixed versions as minimums accordingly and expanded their maximums.

I set a snowflake connector upper bound of <2.1 as they seem to periodically make dangerous breaking changes (though, somehow 1.x -> 2.x was fine?) and I'd prefer not to be caught by surprise again. We can revert that part if we want to leave it unbounded, I don't have very strong feelings on it.

Networkx minimum version set to 2.3, dropped 1.x support
 - removed a backwards compatibility fix
Give colorama a range of supported versions, preserving the old minimum
Give sqlparse a range of supported versions, preserving the old minimum
Bump werkzeug to 0.15.x, 0.16 also works so allow that
 - when 1.0 releases it looks like it will work as well, but wait on that
 - drop 0.14.x support
Set an upper bound on snowflake-connector-python of 2.0.x
@cla-bot cla-bot bot added the cla:yes label Oct 8, 2019
@beckjake beckjake removed the request for review from drewbanin October 8, 2019 17:10
@beckjake beckjake merged commit c1b7c4c into dev/louisa-may-alcott Oct 8, 2019
@beckjake beckjake deleted the feature/upgrade-deps branch October 8, 2019 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade werkzeug Drop support for networkx 1.x
2 participants