Skip to content

ci: fix unmanaged_dependency_check module install - #12812

Merged
chingor13 merged 4 commits into
mainfrom
fix-unmanaged-dep-check
Apr 15, 2026
Merged

ci: fix unmanaged_dependency_check module install#12812
chingor13 merged 4 commits into
mainfrom
fix-unmanaged-dep-check

Conversation

@chingor13

Copy link
Copy Markdown
Contributor

Fixes #12704

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@chingor13
chingor13 force-pushed the fix-unmanaged-dep-check branch from 87f4c28 to 3c53146 Compare April 15, 2026 17:47
@chingor13
chingor13 marked this pull request as ready for review April 15, 2026 18:06
@chingor13
chingor13 requested a review from a team as a code owner April 15, 2026 18:06
@chingor13
chingor13 requested a review from blakeli0 April 15, 2026 18:06

@meltsufin meltsufin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Don't we already have it here: .github/workflows/versions.yaml?
There should probably just be one across the entire repo.

@chingor13

Copy link
Copy Markdown
Contributor Author

Don't we already have it here: .github/workflows/versions.yaml? There should probably just be one across the entire repo.

versions.yaml check is trying to check that the release-please release PR doesn't have any -SNAPSHOT versions or if an existing release pom version already exists.

The unmanaged-dependency-check is for dependencyManagement versions that should be in shared-dependencies

@chingor13
chingor13 merged commit bea5acd into main Apr 15, 2026
110 checks passed
@chingor13
chingor13 deleted the fix-unmanaged-dep-check branch April 15, 2026 21:28
Neenu1995 pushed a commit that referenced this pull request Apr 17, 2026
Fixes #12704

---------

Co-authored-by: cloud-java-bot <cloud-java-bot@google.com>
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.

[sdk-platform-java] Unmanaged dependency check is failing

4 participants