Skip to content
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

Feature request: Stop parsing args after -- #11547

Closed
Dialga opened this issue Jul 29, 2021 · 13 comments · Fixed by #25605
Closed

Feature request: Stop parsing args after -- #11547

Dialga opened this issue Jul 29, 2021 · 13 comments · Fixed by #25605
Assignees
Labels
feat new feature (which has been agreed to/accepted)
Milestone

Comments

@Dialga
Copy link

Dialga commented Jul 29, 2021

For example I want to pass args to the deno repl for testing so it would be helpful to stop parsing args after -- is detected.
deno repl -- --help --xyz

@getspooky
Copy link
Contributor

I think the current error return error: Found argument '--help' which wasn't expected, or isn't valid in this context makes sense

@felipecrs
Copy link

felipecrs commented Aug 23, 2021

@getspooky I think the goal is to test the Deno.args behavior when using deno repl. For example:

$ deno repl -- arg1 arg2
Deno 1.13.1
exit using ctrl+d or close()
> Deno.args
["arg1", "arg2"]

I believe it's a valid use case, and I was about to open such issue as suggestion too.

@stale
Copy link

stale bot commented Oct 22, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Oct 22, 2021
@felipecrs
Copy link

Please don't let it die! :(

@bartlomieju bartlomieju removed the stale label Oct 23, 2021
@stale
Copy link

stale bot commented Dec 22, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Dec 22, 2021
@crowlKats crowlKats removed the stale label Dec 22, 2021
@felipecrs
Copy link

Please don't let this die!

@stale
Copy link

stale bot commented Feb 20, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Feb 20, 2022
@bartlomieju bartlomieju added needs investigation requires further investigation before determining if it is an issue or not and removed stale labels Feb 20, 2022
@stale
Copy link

stale bot commented Apr 23, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Apr 23, 2022
@felipecrs
Copy link

I think I will try to implement it in the coming days.

@stale stale bot removed the stale label Apr 24, 2022
@stale
Copy link

stale bot commented Jun 24, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jun 24, 2022
@Dialga
Copy link
Author

Dialga commented Jun 25, 2022

bump

@stale stale bot removed the stale label Jun 25, 2022
@crowlKats crowlKats added suggestion suggestions for new features (yet to be agreed) and removed needs investigation requires further investigation before determining if it is an issue or not labels Jun 26, 2022
@vegerot
Copy link
Contributor

vegerot commented Apr 7, 2023

This is also blocking easy development for https://github.com/teaxyz/cli

vegerot added a commit to vegerot/cli-1 that referenced this issue Apr 7, 2023
Summary:
Reference blockers for quickly developing tea.
denoland/deno#11547 and
denoland/deno#18631
vegerot added a commit to vegerot/cli-1 that referenced this issue Apr 7, 2023
Summary:
Reference blockers for quickly developing tea.
denoland/deno#11547 and
denoland/deno#18631
mxcl pushed a commit to pkgxdev/pkgx that referenced this issue Apr 7, 2023
Summary:
Reference blockers for quickly developing tea.
denoland/deno#11547 and
denoland/deno#18631
mxcl pushed a commit to pkgxdev/pkgx that referenced this issue Apr 7, 2023
Summary:
Reference blockers for quickly developing tea.
denoland/deno#11547 and
denoland/deno#18631
@bartlomieju bartlomieju added this to the 2.0.0 milestone Aug 31, 2024
@bartlomieju bartlomieju added feat new feature (which has been agreed to/accepted) and removed suggestion suggestions for new features (yet to be agreed) maybe 2.0 labels Aug 31, 2024
@lucacasonato
Copy link
Member

@crowlKats So it seems this si already supported for deno test and deno bench. Only for deno repl is it not supported.

Can you make deno repl -- foobar work?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
feat new feature (which has been agreed to/accepted)
Projects
None yet
7 participants