Skip to content

Support pnpm command for jfrog-cli - #3390

Merged
fluxxBot merged 13 commits into
masterfrom
RTECO-935
Mar 31, 2026
Merged

Support pnpm command for jfrog-cli#3390
fluxxBot merged 13 commits into
masterfrom
RTECO-935

Conversation

@fluxxBot

@fluxxBot fluxxBot commented Mar 11, 2026 •
edited
Loading

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

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
@fluxxBot fluxxBot mentioned this pull request Mar 11, 2026
Merged
5 tasks done
agrasth
agrasth reviewed Mar 16, 2026
View reviewed changes
Comment thread .github/workflows/npmTests.yml Outdated
env:
YARN_IGNORE_NODE: 1
run: go test -v github.com/jfrog/jfrog-cli --timeout 0 --test.npm

@agrasth agrasth Mar 16, 2026

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.

  • Just a reminder to remove.

Comment thread buildtools/cli.go Resolved
Comment thread buildtools/cli.go Resolved
Comment thread .gitignore Resolved
@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
@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 24, 2026
@github-actions

github-actions Bot commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

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

@fluxxBot
fluxxBot requested a review from agrasth March 24, 2026 14:19
bhanurp
bhanurp requested changes Mar 30, 2026
View reviewed changes
Comment thread buildtools/cli.go Resolved
Comment thread docs/buildtools/pnpmcommand/help.go Resolved
Comment thread utils/tests/utils.go Resolved
Comment thread .gitignore Resolved
Comment thread .github/workflows/pnpmTests.yml Resolved
@fluxxBot
fluxxBot requested a review from bhanurp March 30, 2026 12:05
@fluxxBot fluxxBot removed the safe to test Approve running integration tests on a pull request label Mar 30, 2026
@fluxxBot fluxxBot added the safe to test Approve running integration tests on a pull request label Mar 30, 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 31, 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 31, 2026
@fluxxBot
fluxxBot merged commit 0b9ec31 into master Mar 31, 2026
77 of 78 checks passed
@fluxxBot
fluxxBot deleted the RTECO-935 branch March 31, 2026 10:11
@BrewTestBot BrewTestBot mentioned this pull request Apr 1, 2026
Merged
@blinkagent blinkagent Bot mentioned this pull request Aug 24, 2026
Closed

This branch had an error being deployed

1 failed deployment
frogbot — ab94e074 Deployed Mar 31, 2026 by fluxxBot via scan-pull-request #3892
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bhanurp bhanurp bhanurp approved these changes
@agrasth agrasth agrasth approved these changes

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@fluxxBot @bhanurp @agrasth