Skip to content

feat: CLI_ARGS_LIST global variable #2140

New issue

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

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

Already on GitHub? # to your account

Merged
merged 1 commit into from
May 1, 2025
Merged

feat: CLI_ARGS_LIST global variable #2140

merged 1 commit into from
May 1, 2025

Conversation

pd93
Copy link
Member

@pd93 pd93 commented Mar 26, 2025

Fixes #2139
Fixes #2138

Adds a new CLI_ARGS_LIST global variable which contains the pre-dash arguments in an array.

@pd93 pd93 requested review from vmaerten and andreynering March 26, 2025 21:51
@pd93 pd93 force-pushed the split-cli-args-experiment branch from 9da7bb9 to 7674771 Compare April 22, 2025 02:45
@pd93 pd93 changed the title feat: split cli args experiment feat: CLI_ARGS_LIST global variable Apr 22, 2025
Copy link
Member

@vmaerten vmaerten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!
Maybe worth it to wait #2189 and add it :D

@pd93
Copy link
Member Author

pd93 commented May 1, 2025

@vmaerten Just gunna merge this now and we can circle back on the current release items if #2189 gets merged.

@pd93 pd93 merged commit 6896acc into main May 1, 2025
14 checks passed
@pd93 pd93 deleted the split-cli-args-experiment branch May 1, 2025 17:43
pd93 added a commit that referenced this pull request May 1, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a CLI_ARGS_LIST global variable Using for with CLI_ARGS unexpectedly splits arguments containing spaces
2 participants