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

Consider changing exit code of deno task (no args) from 1 to 0 #25632

Closed
kt3k opened this issue Sep 14, 2024 · 0 comments · Fixed by #25637
Closed

Consider changing exit code of deno task (no args) from 1 to 0 #25632

kt3k opened this issue Sep 14, 2024 · 0 comments · Fixed by #25637

Comments

@kt3k
Copy link
Member

kt3k commented Sep 14, 2024

Currently deno task without arguments returns status code 1 even when it successfully lists the tasks.

Is this behavior intentional?

There was a community feedback that this behavior is confusing, and it should return 0.

Note: npm run (with no arg) returns exit code 0.
littledivy added a commit to littledivy/deno that referenced this issue Sep 14, 2024
@littledivy
fix(cli): deno task exit with status 0
f0389f5 
denoland#25632

Exit code 1 indiciates some sort of failure but `deno task` (without
arguments) is used to list available commands.
littledivy added a commit to littledivy/deno that referenced this issue Sep 14, 2024
@littledivy
fix(cli): deno task exit with status 0
703e8dc 
denoland#25632

Exit code 1 indiciates some sort of failure but `deno task` (without
arguments) is used to list available commands.
@littledivy littledivy mentioned this issue Sep 14, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(cli): deno task exit with status 0 littledivy/deno
1 participant
@kt3k