-
Notifications
You must be signed in to change notification settings - Fork 125
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
Update dependencies, fix conflicts with other packages #764
Update dependencies, fix conflicts with other packages #764
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
Submitting this PR as there are conflicts with Snowflake's dependency on the older I would humbly request/suggest that this package drop the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Please update the branch. |
Changes arrow to v11, updates all dependencies, and removes replace statements within go.mod, as these cause conflicts/issues when importing this package.
4742946
to
77919ad
Compare
@zeroshade done. |
@sfc-gh-igarish branch is updated! please merge at your earliest convenience! |
Still waiting for test to finish and pass. |
@zeroshade I would assume that's because this is PR, and those secrets are available for PRs. |
ok. I get your PR and created a new PR on top of it and link with your PR. Let's see how tests goes now here. |
Description
Changes arrow to v11, updates all dependencies, and removes replace statements within go.mod, as these cause conflicts/issues when importing this package.
Checklist
make fmt
to fix inconsistent formatsmake lint
to get lint errors and fix all of them