Skip to content

Support pnpm command for jfrog-cli#3390

Open
fluxxBot wants to merge 9 commits intomasterfrom
RTECO-935
Open

Support pnpm command for jfrog-cli#3390
fluxxBot wants to merge 9 commits intomasterfrom
RTECO-935

Conversation

@fluxxBot
Copy link
Contributor

@fluxxBot fluxxBot commented Mar 11, 2026

  • 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 ./....

Description:
Support pnpm commands

Depends on:

  1. Support pnpm command jfrog-cli-artifactory#393

@fluxxBot fluxxBot added the new feature Automatically generated release notes label Mar 11, 2026
@fluxxBot fluxxBot changed the title RTECO-935 - Support pnpm command for jfrog-cli Support pnpm command for jfrog-cli Mar 11, 2026
env:
YARN_IGNORE_NODE: 1
run: go test -v github.com/jfrog/jfrog-cli --timeout 0 --test.npm

Copy link
Contributor

Choose a reason for hiding this comment

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

  • Just a reminder to remove.

**/testdata/**/bin
testdata/**/.gradle

.cursor No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

You can remove this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

what exactly you want me to remove?

@fluxxBot fluxxBot added the safe to test Approve running integration tests on a pull request label Mar 17, 2026
@fluxxBot fluxxBot added safe to test Approve running integration tests on a pull request and removed safe to test Approve running integration tests on a pull request labels Mar 17, 2026
@github-actions
Copy link
Contributor

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


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new feature Automatically generated release notes safe to test Approve running integration tests on a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants