Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rm-empty folders with path not ends with '/' removes the entire path #149

Closed
dantran opened this issue Oct 1, 2024 · 2 comments
Closed

Comments

@dantran
Copy link

dantran commented Oct 1, 2024

jf rm-empty folders dps-maven-staging/com
12:40:30 [🔵Info] Searching for all items under dps-maven-staging/com
12:40:30 [🔵Info] Searching artifacts...
12:40:31 [🔵Info] Searching for items in Artifactory...
12:40:32 [🔵Info] Found 1 artifact.
12:40:32 [🔵Info] Found 1 empty folder.
  dps-maven-staging/com/
Are you sure you want to delete the above paths? (y/n) [n]? n

if I hit yes, it will move my 'com' folder including of of its non empty folders to trash

This is a severe bug

versions:

jf --version
jf version 2.63.0

jf rm-empty --version
rm-empty version v1.0.4
@eyalbe4
Copy link
Contributor

eyalbe4 commented Oct 2, 2024

@dantran,
It is best to post issues on the code repository that hosts the actual plugin and close this issue.
This repository merely serves as the registry for the plugin. Also, creating a pull request with the fix is the recommend approach.

@dantran dantran closed this as completed Oct 2, 2024
@dantran
Copy link
Author

dantran commented Oct 2, 2024

see jfrog/jfrog-cli-plugins#102

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

No branches or pull requests

2 participants