Skip to content

HMS-10989: Update Tang for version duplication fix - #1593

Merged
rverdile merged 1 commit into
content-services:mainfrom
rverdile:maven-versions-get-refactor
Jul 22, 2026
Merged

HMS-10989: Update Tang for version duplication fix#1593
rverdile merged 1 commit into
content-services:mainfrom
rverdile:maven-versions-get-refactor

Conversation

@rverdile

Copy link
Copy Markdown
Contributor

Summary

Tang was returning duplicated versions in latest_releases. It has also been refactored to replace MavenBuildList with MavenVersionsList, which more closely resembles the use-case for the method. These changes update to the new version of Tang. It also updates the create_maven_repo.sh script to use a fixture with curated metadata to test these fixes.

Testing steps

  1. Checkout Tang PR: HMS-10989: update maven query to consider version without release tang#43
  2. Use ./scripts/create_maven_repo.sh to import new repos
  3. Check the maven PackagesList, VersionsList, and VersionDetails APIs for the expected Version and Release information.

@xbhouse

xbhouse commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Tang was returning duplicated versions in latest_releases. It has also
been refactored to replace MavenBuildList with MavenVersionsList, which
more closely resembles the use-case for the method. These changes update
to the new version of a Tang. It also updates the create_maven_repo
script to use a fixture with curated metadata to test these fixes.
@rverdile
rverdile force-pushed the maven-versions-get-refactor branch from e47f931 to e434f88 Compare July 22, 2026 13:12

@TenSt TenSt 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.

lgtm!

@TenSt TenSt self-assigned this Jul 22, 2026
@rverdile
rverdile merged commit be1bf71 into content-services:main Jul 22, 2026
20 checks passed
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.

3 participants