jfrog / jfrog-cli-core

Apache License 2.0

33 stars 58 forks source link

Fix npm publish with provided tarball #1162

Closed EyalDelarea closed 6 months ago

EyalDelarea commented 6 months ago

[x] All tests passed. If this feature is not already covered by the tests, I added new tests.
[x] All static analysis checks passed.
[x] This pull request is on the dev branch.
[x] I used gofmt for formatting the code before submitting the pull request.

Fix npm pack when reading a pre packed npm package

When reading for an already packed npm package, there was a missing file path that wasn't set because of the changes to handle workspaces. This PR should fix it by setting the file path.

github-actions[bot] commented 6 months ago

[![👍 Frogbot scanned this pull request and found that it did not add vulnerable dependencies.](https://raw.githubusercontent.com/jfrog/frogbot/master/resources/v2/noVulnerabilityBannerPR.png)](https://docs.jfrog-applications.jfrog.io/jfrog-applications/frogbot)

[🐸 JFrog Frogbot](https://docs.jfrog-applications.jfrog.io/jfrog-applications/frogbot)

jfrog / jfrog-cli-core

Fix npm publish with provided tarball #1162

[x] I used gofmt for formatting the code before submitting the pull request.

Fix npm pack when reading a pre packed npm package