Skip to content

Run tests with latest Ember CLI#134

Merged
simonihmig merged 1 commit intomainfrom
update-embercli
Jul 6, 2023
Merged

Run tests with latest Ember CLI#134
simonihmig merged 1 commit intomainfrom
update-embercli

Conversation

@simonihmig
Copy link
Copy Markdown
Collaborator

@simonihmig simonihmig commented Jul 2, 2023

Use canary version of ember-cli on master branch, until we get a stable/beta release on npm, which includes better support of --typescript (PR). Cleans up previous weirdness.

@simonihmig simonihmig force-pushed the update-embercli branch 2 times, most recently from a4524fd to 9675762 Compare July 2, 2023 21:23
@simonihmig
Copy link
Copy Markdown
Collaborator Author

TS tests failing due to #133

Comment thread tests/cli.test.ts Outdated
let cwd = '';
let tmpDir = '';
let distDir = '';
describe(`--typescript with ${packageManager}`, () => {
Copy link
Copy Markdown
Collaborator Author

@simonihmig simonihmig Jul 2, 2023

Choose a reason for hiding this comment

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

Basically just white-space changes here due to putting this into previous describe() that runs for every package manager. Best reviewed with Hide whitespace enabled!

@simonihmig simonihmig marked this pull request as ready for review July 3, 2023 20:22
@simonihmig simonihmig requested a review from NullVoxPopuli July 3, 2023 20:23
Use canary version of ember-cli on master branch, until we ge get a stable/beta release on npm, which includes better support of `--typescript` ([PR](ember-cli/ember-cli#10283)). Cleans up previous weirdness.
@simonihmig simonihmig merged commit c92bf61 into main Jul 6, 2023
@simonihmig simonihmig deleted the update-embercli branch July 6, 2023 09:55
aklkv pushed a commit to aklkv/addon-blueprint that referenced this pull request Feb 8, 2026
aklkv pushed a commit to aklkv/addon-blueprint that referenced this pull request Feb 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants