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.

Sign up for GitHub

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

Jump to bottom

[Feature Request]: Hop Run - Specify a custom separator char for parameters #2923

Closed
sramazzina opened this issue May 12, 2023 · 0 comments · Fixed by #2924
Closed

[Feature Request]: Hop Run - Specify a custom separator char for parameters #2923

sramazzina opened this issue May 12, 2023 · 0 comments · Fixed by #2924
Labels
Hop Run new feature P3 Nice to have
Milestone
2.5

Comments

@sramazzina
Copy link
Contributor

What would you like to happen?

Add a new switch called parameter separator (--parameter-separator or -ps) to specify the char we use to separate the various parameters. Default value as parameter separator is the comma, as it is now

Issue Priority

Priority: 3

Issue Component

Component: Hop Run
@github-actions github-actions bot added P3 Nice to have Hop Run labels May 12, 2023
@sramazzina sramazzina changed the title [Feature Request]: Hop Run - Specify a separator char for parameters [Feature Request]: Hop Run - Specify a custom separator char for parameters May 12, 2023
@sramazzina sramazzina mentioned this issue May 12, 2023
Merged
5 tasks
@sramazzina sramazzina added this to the 2.5 milestone May 12, 2023
hansva added a commit that referenced this issue May 17, 2023
@hansva
Merge pull request #2924 from sramazzina/2923
ffee560 
fix #2923
hansva added a commit to nadment/hop that referenced this issue May 17, 2023
@hansva
Small cleanup for apache#2000, and small bugfix for apache#2923
af135c5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Hop Run new feature P3 Nice to have
Projects
None yet
Milestone
2.5
Development

Successfully merging a pull request may close this issue.

fix #2923 sramazzina/hop
2 participants
@hansva @sramazzina