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: remove dep hacks for old cycle deps #2908

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

codyoss
Copy link
Member

@codyoss codyoss commented Dec 10, 2024

I believe our dep cycle issues should have been fixed in #2907.
Verified issue in #2543 is fixed with this change.

Fixes: #2559

I believe our dep cycle issues should have been fixed in googleapis#2907.
@codyoss codyoss requested a review from a team as a code owner December 10, 2024 16:44
@codyoss codyoss added the automerge Merge the pull request once unit tests and other checks pass. label Dec 10, 2024
Copy link
Member

@quartzmo quartzmo left a comment

Choose a reason for hiding this comment

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

Nice cleanup!

@gcf-merge-on-green gcf-merge-on-green bot merged commit f4c1ac6 into googleapis:main Dec 10, 2024
4 checks passed
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Dec 10, 2024
@codyoss codyoss deleted the cleanup1210 branch December 10, 2024 17:02
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.

deps: remove forced dependency on main google-cloud-go module
2 participants