Skip to content
New issue

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

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

Already on GitHub? # to your account

Move sqlalchemy to snowflake optional dep #102

Closed
korikuzma opened this issue Oct 21, 2024 · 2 comments
Closed

Move sqlalchemy to snowflake optional dep #102

korikuzma opened this issue Oct 21, 2024 · 2 comments

Comments

@korikuzma
Copy link
Contributor

korikuzma commented Oct 21, 2024

I'm not sure if we needed to add sqlalchemy to required dependencies (since not everyone will be using sqlalchemy). I think the optional snowflake dep handles this?

Originally posted by @korikuzma in #101 (comment)

Will need to update docs to show optional deps and how to install

@jsstevenson
Copy link
Contributor

(It would also need to be in the Postgres group)

@korikuzma
Copy link
Contributor Author

@jsstevenson Oh that's my bad for not being familiar with the source code. I didn't realize we were using sqlalchemy with the postgres backend. I just assumed we were only using psycopg2. I'll close

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants