-
Notifications
You must be signed in to change notification settings - Fork 59
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
Support Gradle args with spaces #848
Support Gradle args with spaces #848
Conversation
keeganjm-bsc
commented
Jun 27, 2023
•
edited
Loading
edited
- All tests passed. If this feature is not already covered by the tests, I added new tests.
- All static analysis checks passed.
- This pull request is on the dev branch.
- I used gofmt for formatting the code before submitting the pull request.
@yahavi @eyalb4doc @RobiNino Please approve this for the Frogbot Scan job so this PR can move forward. |
@yahavi @eyalbe4 @RobiNino please review and merge. This PR is a dependency for another PR IN Jfrog-cli project here jfrog/jfrog-cli#2029. We have been trying to get these changes merged for over 2 weeks now. Please review and merge these so we can move forward with this. |
|
I checked this change on the JFrog CLI and I'm not conviced it solves the issue
@yahavi can you elaborate more on this? It is similar to the changes that were required for mvn command. |
@keeganjm-bsc I elaborated in pull requests at the JFrog CLi repository: jfrog/jfrog-cli#2029 (comment) Please let me know what you think. |
@keeganjm-bsc, |