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

fix: Fixed Browser PKG #626

Merged
merged 1 commit into from
Aug 1, 2021

Conversation

ChrisIsidora
Copy link
Contributor

Attempt to fix #624 (comment)

@ChrisIsidora ChrisIsidora requested a review from a team as a code owner July 31, 2021 07:09
@ChrisIsidora ChrisIsidora requested a review from alldoami July 31, 2021 07:09
Comment on lines -231 to +213
github.com/chanzuckerberg/go-misc v0.0.0-20210727205931-6daf95f6727e h1:LyRaq/Sz1omD+bBuVwZBVxAv3qEUHAzgol5IFdG1YtY=
github.com/chanzuckerberg/go-misc v0.0.0-20210727205931-6daf95f6727e/go.mod h1:mQwH9ExhxX/18odRhummSPc21vJ7TqB5y8u4uYv+1h4=
github.com/cheggaaa/pb v1.0.27/go.mod h1:pQciLPpbU0oxA0h+VJYYLxO+XeDQb5pZijXscXHm81s=
Copy link
Contributor Author

@ChrisIsidora ChrisIsidora Jul 31, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alldoami Is you run go mod graph you could see that go-misc v0.0.0-20210727205931-6daf95f6727e added the browser dependency that is breaking the release?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you explain how this works for future reference 😅

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alldoami I also don't know. All I know is that when the go mods where updated the go-misc mod of chanzuckerberg was also updated and in the new updated version it's pulling a new version of the indirect dependency browser pkg which broke the release. Using go mod graph you can find out where the indirect dependency is coming from, which in this case was flgo-misc

@alldoami
Copy link
Contributor

alldoami commented Aug 1, 2021

/ok-to-test sha=c0bb8c1

@github-actions
Copy link

github-actions bot commented Aug 1, 2021

Integration tests success for c0bb8c1

@alldoami alldoami merged commit 4b213d1 into Snowflake-Labs:main Aug 1, 2021
@alldoami
Copy link
Contributor

alldoami commented Aug 1, 2021

Thanks so much @ChrisIsidora !

@ChrisIsidora ChrisIsidora deleted the fix/browser-pkg-fix branch August 2, 2021 07:26
jtzero pushed a commit to rxrevu/terraform-provider-snowflake that referenced this pull request Aug 19, 2021
anton-chekanov pushed a commit to anton-chekanov/terraform-provider-snowflake that referenced this pull request Jan 25, 2022
daniepett pushed a commit to daniepett/terraform-provider-snowflake that referenced this pull request Feb 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants