changed to enable to use command line arguments supplied for the script.... #298
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…pt. And to make include the script name in error message, change GroovyShell.evaluate(String) to GroovyShell.run(File).
Member
|
Closing outdated pull requests. |
jdaugherty
pushed a commit
to jdaugherty/grails-core
that referenced
this pull request
Jan 15, 2025
Update dependency org.springframework.boot:spring-boot-test to v2.7.14
jdaugherty
pushed a commit
to jdaugherty/grails-core
that referenced
this pull request
Apr 14, 2025
…plugin to v2.0.1 (apache#298) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
jdaugherty
pushed a commit
to jdaugherty/grails-core
that referenced
this pull request
Jun 28, 2025
This pull request updates the release-drafter template to enhance the quality of release notes. Additionally, we've introduced two new triggers, `pull_request` and `pull_request_target`, which automatically apply labels to pull requests based on their titles. This is essential for autolabeling and to accommodate pull requests from forks.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
... And to make include the script name in error message, change GroovyShell.evaluate(String) to GroovyShell.run(File).
Use of shell variable 'args' in groovy scripts are very common and useful. But in current implementation, scripts called from 'run-script' can't use args at all so fixed it.
As a side effect, in this version can't spcify multiple scripts at one command invocation of run-script.
It might be better to create another new script(for example 'run-batch') to use 'args' arguments from script.
I have no good idea anymore so send in this form as a one proposal.
Best regards,