-
-
Notifications
You must be signed in to change notification settings - Fork 3
Import tests from @embroider/addon-blueprint #26
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.
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add Lints
Fix test timeouts
As suggested in embroider-build/addon-blueprint#8 (comment). Closes #8
Move .md files to published package
…an ignore all errors under pnpm
Add support for npm and pnpm
Add Typescript support
Closes #86
Support existing monorepos
Split monorepo tests in CI
Add ESLint rule to require that relative imports use full extensions
Bump pnpm/action-setup to v4
* ember-cli < 6.1 is too old * Switch to ESLint 9 * fix * afix * fix * ope * Cleanup, update prettier config * ope * Fix lints * Lints * lints * Fix assertions * Use TypeScript 5.6 * Upgrade typescript-eslint * Set project: true * Override ember-cli's tsconfig.json see: ember-cli/ember-cli#10611 and ember-cli/ember-cli#10613 and ember-cli/ember-cli#10612 * Re-lock
Remove pnpm version from pnpm/action-setup
Remove remaining pnpm version from pnpm/action-setup
Default new github repos don't have write contents permissions on github tokens
Add permissions to the push-dist workflow
NullVoxPopuli
approved these changes
Apr 8, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This imports the tests from
@embroider/addon-blueprint
and keeps their history 🎉I've also started to simplify a few things since we don't need it to be quite so complicated 🙈
Note: not all the tests are running in this branch yet. We can either merge it and enabled (or delete) the tests as we go or we can try to fix more before merging