Skip to content

Support a --ts alias for the addon, init and new commands#10804

Merged
mansona merged 1 commit intoember-cli:masterfrom
bertdeblock:ts-alias
Sep 10, 2025
Merged

Support a --ts alias for the addon, init and new commands#10804
mansona merged 1 commit intoember-cli:masterfrom
bertdeblock:ts-alias

Conversation

@bertdeblock
Copy link
Member

@bertdeblock bertdeblock commented Sep 9, 2025

We do the same for the generate and destroy commands.

This work is supported by the Bert De Block Initiative.

@mansona
Copy link
Member

mansona commented Sep 10, 2025

Nice one! @bertdeblock do you happen to know if this just works with the new app blueprint? i.e. do we need to update https://github.com/ember-cli/ember-app-blueprint to have the same alias? or is it just a concern of the ember-cli command itself?

@bertdeblock
Copy link
Member Author

bertdeblock commented Sep 10, 2025

No blueprint updates are required AFAICT. ember-cli should translate --ts to options.typescript correctly. But I'll double check for you, just a sec!

@bertdeblock
Copy link
Member Author

bertdeblock commented Sep 10, 2025

Here ya go!

➜ node bin/ember addon foo --ts --skip-install --skip-git
{
  dryRun: false,
  verbose: false,
  blueprint: 'addon',
  skipNpm: true,
  skipGit: true,
  lintFix: true,
  ciProvider: 'github',
  typescript: true,
  strict: false,
  checkForUpdates: false
}

➜ node bin/ember addon foo --ts=false --skip-install --skip-git
{
  dryRun: false,
  verbose: false,
  blueprint: 'addon',
  skipNpm: true,
  skipGit: true,
  lintFix: true,
  ciProvider: 'github',
  typescript: false,
  strict: false,
  checkForUpdates: false
}

mansona
mansona approved these changes Sep 10, 2025
View reviewed changes
Copy link
Member

@mansona mansona left a comment

Choose a reason for hiding this comment

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

LGTM

@mansona mansona merged commit 39d0f60 into ember-cli:master Sep 10, 2025
70 of 71 checks passed
This was referenced Sep 10, 2025
Merged
Merged
@mansona mansona mentioned this pull request Oct 14, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mansona mansona mansona approved these changes

Assignees

No one assigned

Labels

enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bertdeblock @mansona