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

use latest versions of dependencies #423

Closed
cmcarthur opened this issue May 11, 2017 · 3 comments
Closed

use latest versions of dependencies #423

cmcarthur opened this issue May 11, 2017 · 3 comments
Labels
enhancement New feature or request
Milestone

Comments

@cmcarthur
Copy link
Member

psycopg2 > 2.7.1?
sqlparse > 0.2.3?
csvkit > 1.0.2?
celery? do we even use this?
voluptuous > 0.10.5
snowflake-connector-python > 1.3.16

there are other, non-explicit dependencies that should be upgraded:

cryptography > 1.8.1

@cmcarthur cmcarthur added the enhancement New feature or request label May 11, 2017
@cmcarthur cmcarthur modified the milestone: 0.8.2 Release May 12, 2017
@drewbanin
Copy link
Contributor

drewbanin commented May 13, 2017

Changelogs:

  • psycopg2
  • sqlparse
  • csvkit
    • this is a major upgrade with some breaking changes to the CLI -- need to evaluate how that impacts the API
  • celery - we definitely don't use this directly -- it's a dep of the snowplow module. I'll investigate if we can remove it (I think we can)
  • voluptuous
  • snowflake-connector-python

@cmcarthur i don't wholly understand the cryptography note -- should we add that to install_requires?

@drewbanin
Copy link
Contributor

We can remove the celery dep if we upgrade snowplow-tracker to 0.8.0 but it looks like there's a dep on six=1.9.0 which makes things hard. Let's leave the celery requirement for now and revisit in the future.

@drewbanin
Copy link
Contributor

Looks like csvkit's api is deprecated and we should use agate instead? Let's leave this alone for now

Help on package csvkit:

NAME
    csvkit

DESCRIPTION
    This module contains csvkit's superpowered alternative to the standard Python
    CSV reader and writer. It can be used as a drop-in replacement for the standard
    module.

    .. warn::

        Since version 1.0 csvkit relies on `agate <http://agate.rtfd.org>`_'s
    CSV reader and writer. This module is supported for legacy purposes only and you
    should migrate to using agate.

PACKAGE CONTENTS
...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants