Skip to content

Fixed build info recording npm workspaces - #3124

Merged
fluxxBot merged 1 commit into
jfrog:masterfrom
fluxxBot:npmworkspaces
Sep 24, 2025
Merged

fluxxBot merged 1 commit into
jfrog:masterfrom
fluxxBot:npmworkspaces

Conversation

@fluxxBot

@fluxxBot fluxxBot commented Sep 22, 2025 •

Copy link
Copy Markdown
Contributor
  • All tests have passed. If this feature is not already covered by the tests, new tests have been added.
  • The pull request is targeting the master branch.
  • The code has been validated to compile successfully by running go vet ./....
  • The code has been formatted properly using go fmt ./....

Problem -> npm publish --workspaces recorded only the last published artifact in the modules due to overriding of the artifactReader

solution -> instead of using a single reader and overriding it, use slice of readers

depends on:

  1. Fixed build info recording npm workspaces jfrog-cli-artifactory#168

@fluxxBot fluxxBot added the bug Something isn't working label Sep 22, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Sep 22, 2025
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Sep 22, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Sep 22, 2025
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Sep 22, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Sep 23, 2025
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Sep 23, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Sep 24, 2025
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Sep 24, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Sep 24, 2025
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Sep 24, 2025
@ehl-jf ehl-jf added the safe to test Approve running integration tests on a pull request label Sep 24, 2025
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Sep 24, 2025
@fluxxBot
fluxxBot enabled auto-merge (squash) September 24, 2025 12:42
@github-actions

Copy link
Copy Markdown
Contributor

👍 Frogbot scanned this pull request and did not find any new security issues.


@fluxxBot
fluxxBot merged commit 8808261 into jfrog:master Sep 24, 2025
66 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants