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.

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

Updated blueprints to match @embroider/addon-blueprint@2.2.0 #62

Merged
merged 4 commits into from
Aug 12, 2023

Conversation

ijlee2
Copy link
Owner

@ijlee2 ijlee2 commented Aug 12, 2023

Description

I downstreamed changes from @embroider/addon-blueprint@2.2.0, 2.2.1, and 2.3.0.

Notes

  1. @embroider/addon-blueprint@2.2.0 adds support for *.{gjs,gts} files. To keep the scope small, I did not include this change.

  2. @embroider/addon-dev@4.1.0 may introduce breaking changes to the codemod. That is, there may be files that the codemod needs to update, in order for the end-developer's project to run with @embroider/addon-dev@4.1.0. I didn't investigate yet if such files exist.

    To account for possible bugs, I'll make the minor release 0.10.0 instead of a patch release 0.9.2.

@ijlee2 ijlee2 added enhance: code Issue asks for new feature or refactor breaking Breaking change will result in a new major version labels Aug 12, 2023
@@ -23,7 +23,7 @@
"concurrently": "<%= context.projectRoot.devDependencies['concurrently'] %>"
}
}<% } else if (options.packageManager.isPnpm) { %>{
"name": "<%= options.packages.addon.name %>",
"name": "workspace-root",
Copy link
Owner Author

Choose a reason for hiding this comment

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

By giving the workspace root a name, we may easily distinguish where dependencies come from, when we run the outdated command, e.g. pnpm outdated -r.

@ijlee2 ijlee2 marked this pull request as ready for review August 12, 2023 07:59
@ijlee2 ijlee2 changed the title Updated blueprints Updated blueprints to match @embroider/addon-blueprint@2.2.0 Aug 12, 2023
@ijlee2 ijlee2 merged commit 83394c4 into main Aug 12, 2023
2 checks passed
@ijlee2 ijlee2 deleted the update-blueprints branch August 12, 2023 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Breaking change will result in a new major version enhance: code Issue asks for new feature or refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant