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.

Sign up for GitHub

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

Jump to bottom

Couple of misc fixes on deploy #607

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Couple of misc fixes on deploy #607

wants to merge 4 commits into from

Conversation

joshelser
Copy link
Contributor

  • Fail gracefully if not in a git checkout
  • Use correct warehouse kwarg for python connector
  • Pass rolename from snowsql config to role in python connector

@joshelser
Couple of misc fixes on deploy
beefad8 
* Fail gracefully if not in a git checkout
* Use correct `warehouse` kwarg for python connector
* Pass `rolename` from snowsql config to `role` in python connector
@joshelser joshelser requested review from rymurr and robertmaybin April 1, 2024 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rymurr rymurr rymurr approved these changes

@robertmaybin robertmaybin Awaiting requested review from robertmaybin

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@joshelser @rymurr