Skip to content

add help alias to jake --tasks in Jakefile.js#25956

Merged
mhegazy merged 1 commit into
microsoft:masterfrom
fredrb:alias-jake-help-command
Jul 26, 2018
Merged

add help alias to jake --tasks in Jakefile.js#25956
mhegazy merged 1 commit into
microsoft:masterfrom
fredrb:alias-jake-help-command

Conversation

@fredrb
Copy link
Copy Markdown
Contributor

@fredrb fredrb commented Jul 26, 2018

Fixes #25955

DanielRosenwasser
DanielRosenwasser approved these changes Jul 26, 2018
View reviewed changes
Copy link
Copy Markdown
Member

@DanielRosenwasser DanielRosenwasser left a comment

Choose a reason for hiding this comment

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

Pretty indirect compared to changing the README to use jake -T but I guess it's fine :)

@fredrb
Copy link
Copy Markdown
Contributor Author

fredrb commented Jul 26, 2018 •
edited
Loading

I thought about that at first, but I decided to alias --tasks to help for two reasons:

  • help fits the pattern of the rest of commands. There are no tasks (neither parameters) passed with double dashes.
  • --task is completely tied to jake. I think its better (from a UX perspective) to change the underlying command behind help than keep TypeScript's documentation tied to Jake's standard commands.

@mhegazy mhegazy merged commit 3ff667e into microsoft:master Jul 26, 2018
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@DanielRosenwasser DanielRosenwasser DanielRosenwasser approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@fredrb @DanielRosenwasser @mhegazy