-
Notifications
You must be signed in to change notification settings - Fork 143
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
Release prep #1406
Merged
Merged
Release prep #1406
Changes from all commits
Commits
Show all changes
3 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
8 changes: 8 additions & 0 deletions
8
.changeset/automated-from-137fcab566174aad3ebb67dda12ac80572f78ab9.md
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
--- | ||
# Change from: 137fcab566174aad3ebb67dda12ac80572f78ab9 | ||
# | ||
# Each of these should be one of "patch", "minor", "major" | ||
"@embroider/compat": patch | ||
--- | ||
|
||
[#1381](https://github.com/embroider-build/embroider/pull/1381) : add tests for helper name collisions with html elements or js keywords : _by [@void-mAlex](https://github.com/void-mAlex)_ |
7 changes: 7 additions & 0 deletions
7
.changeset/automated-from-21eae416d9fa85ea6f9be55945c44ab3da2c655b.md
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
--- | ||
# Change from: 21eae416d9fa85ea6f9be55945c44ab3da2c655b | ||
# | ||
# Each of these should be one of "patch", "minor", "major" | ||
--- | ||
|
||
[#1364](https://github.com/embroider-build/embroider/pull/1364) : Unstable release sync with main : _by [@NullVoxPopuli](https://github.com/NullVoxPopuli), [@ef4](https://github.com/ef4)_ |
7 changes: 7 additions & 0 deletions
7
.changeset/automated-from-3a232cc04a68f86d0acfa5e7edfa764b990efdbf.md
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
--- | ||
# Change from: 3a232cc04a68f86d0acfa5e7edfa764b990efdbf | ||
# | ||
# Each of these should be one of "patch", "minor", "major" | ||
--- | ||
|
||
[#1401](https://github.com/embroider-build/embroider/pull/1401) : Add relevant changesets for the upcoming release and evaluate their impact : _by [@NullVoxPopuli](https://github.com/NullVoxPopuli)_ |
8 changes: 8 additions & 0 deletions
8
.changeset/automated-from-3a9d8ade05e92b65045a01d59898f063e337fcd1.md
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
--- | ||
# Change from: 3a9d8ade05e92b65045a01d59898f063e337fcd1 | ||
# | ||
# Each of these should be one of "patch", "minor", "major" | ||
"@embroider/compat": minor | ||
--- | ||
|
||
[#1392](https://github.com/embroider-build/embroider/pull/1392) : Allow running publish-unstable on workflow_dispatch : _by [@NullVoxPopuli](https://github.com/NullVoxPopuli)_ |
This file contains 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
7 changes: 7 additions & 0 deletions
7
.changeset/automated-from-514566f036a28113d3aefd4430f256658c08e82e.md
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
--- | ||
# Change from: 514566f036a28113d3aefd4430f256658c08e82e | ||
# | ||
# Each of these should be one of "patch", "minor", "major" | ||
--- | ||
|
||
[#1395](https://github.com/embroider-build/embroider/pull/1395) : Revving yarn.lock : _by [@ef4](https://github.com/ef4)_ |
8 changes: 8 additions & 0 deletions
8
.changeset/automated-from-55b12fea25b68bae28886e7705f9e3637ddf0b55.md
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
--- | ||
# Change from: 55b12fea25b68bae28886e7705f9e3637ddf0b55 | ||
# | ||
# Each of these should be one of "patch", "minor", "major" | ||
"@embroider/compat": minor | ||
--- | ||
|
||
[#1389](https://github.com/embroider-build/embroider/pull/1389) : bump unstable versions by at least a patch : _by [@mansona](https://github.com/mansona)_ |
15 changes: 15 additions & 0 deletions
15
.changeset/automated-from-5d605075b84e902609eccacd6decf7e6c4ca7250.md
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
--- | ||
# Change from: 5d605075b84e902609eccacd6decf7e6c4ca7250 | ||
# | ||
# Each of these should be one of "patch", "minor", "major" | ||
"@embroider/addon-dev": minor | ||
"@embroider/addon-shim": minor | ||
"@embroider/compat": minor | ||
"@embroider/core": minor | ||
"@embroider/macros": minor | ||
"@embroider/shared-internals": minor | ||
"@embroider/util": minor | ||
"@embroider/webpack": minor | ||
--- | ||
|
||
[#1354](https://github.com/embroider-build/embroider/pull/1354) : Add glint helper types for more macros : _by [@vlascik](https://github.com/vlascik)_ |
7 changes: 7 additions & 0 deletions
7
.changeset/automated-from-72d5a8e1edf20d0512cd98d1979b7b92360e6fd0.md
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
--- | ||
# Change from: 72d5a8e1edf20d0512cd98d1979b7b92360e6fd0 | ||
# | ||
# Each of these should be one of "patch", "minor", "major" | ||
--- | ||
|
||
[#1390](https://github.com/embroider-build/embroider/pull/1390) : Change namespace for publish-unstable cancel-in-progress : _by [@backspace](https://github.com/backspace)_ |
7 changes: 7 additions & 0 deletions
7
.changeset/automated-from-77c945c528f8e828276e536d25473946f243337c.md
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
--- | ||
# Change from: 77c945c528f8e828276e536d25473946f243337c | ||
# | ||
# Each of these should be one of "patch", "minor", "major" | ||
--- | ||
|
||
[#1405](https://github.com/embroider-build/embroider/pull/1405) : Upgrade changeset-recover : _by [@NullVoxPopuli](https://github.com/NullVoxPopuli)_ |
This file contains 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
8 changes: 8 additions & 0 deletions
8
.changeset/automated-from-a6cd4aac9b7497991b7b1acc18d0e78dccb8cd78.md
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
--- | ||
# Change from: a6cd4aac9b7497991b7b1acc18d0e78dccb8cd78 | ||
# | ||
# Each of these should be one of "patch", "minor", "major" | ||
"@embroider/util": patch | ||
--- | ||
|
||
[#1388](https://github.com/embroider-build/embroider/pull/1388) : Enable prettier in ci : _by [@NullVoxPopuli](https://github.com/NullVoxPopuli)_ |
7 changes: 7 additions & 0 deletions
7
.changeset/automated-from-bae34674206d6e519e634adcd6f2e26f29030955.md
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
--- | ||
# Change from: bae34674206d6e519e634adcd6f2e26f29030955 | ||
# | ||
# Each of these should be one of "patch", "minor", "major" | ||
--- | ||
|
||
[#1393](https://github.com/embroider-build/embroider/pull/1393) : continue deploying unstable packages even with an error : _by [@mansona](https://github.com/mansona)_ |
7 changes: 7 additions & 0 deletions
7
.changeset/automated-from-df7f4412bd8b1f996377a37bcf70e4c7937fb9c0.md
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
--- | ||
# Change from: df7f4412bd8b1f996377a37bcf70e4c7937fb9c0 | ||
# | ||
# Each of these should be one of "patch", "minor", "major" | ||
--- | ||
|
||
[#1398](https://github.com/embroider-build/embroider/pull/1398) : Add prepare changelog workflow to automatically propose what should be in sync for us : _by [@NullVoxPopuli](https://github.com/NullVoxPopuli)_ |
7 changes: 7 additions & 0 deletions
7
.changeset/automated-from-e05dfdc5496dd5b1aaa2684816c1a4e05c4f88c4.md
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
--- | ||
# Change from: e05dfdc5496dd5b1aaa2684816c1a4e05c4f88c4 | ||
# | ||
# Each of these should be one of "patch", "minor", "major" | ||
--- | ||
|
||
[#1345](https://github.com/embroider-build/embroider/pull/1345) : Update release workflow : _by [@NullVoxPopuli](https://github.com/NullVoxPopuli)_ |
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
--- | ||
'@embroider/util': patch | ||
'@embroider/util': minor | ||
--- | ||
|
||
Improve types of `ensure-safe-component` helper | ||
|
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,60 @@ | ||
# @embroider/core 2.x -> 3.x | ||
|
||
## Breaking changes and how to resolve them | ||
|
||
1. `unsupported ambiguous syntax`: When using `staticComponents` or `staticHelpers`, we no longer tolerate forms that are ambiguous between components, helpers, and this-property-fallback. For example, if | ||
```hbs | ||
{{thing}} | ||
``` | ||
|
||
appears by itself in a (non-strict) template, it could mean: | ||
- a component invocation: `<Thing />` | ||
- a helper invocation: `{{ (thing) }}` | ||
- property-this-fallback: `{{this.thing}}` | ||
|
||
The benefit of this change is that we can get rid of a lot of gnarly compatbility code that reduced build performance and introduced subtle caveats. | ||
|
||
Ember itself no longer supports property-this-fallback as of 4.0. And the Ember default blueprint has shipped with a default lint rule for multiple years that tells you not to use `{{thing}}`, in favor of angle bracket invocation or explicit `{{this.thing}}`. | ||
|
||
If you're hitting this problem in your own code, change the ambiguous form to one of the three above unambiguous forms: a component with angle brackets, a helper with parentheses, or an explicit `this.` property. | ||
|
||
If you're hitting this problem in third-party code that you don't want to patch, you can use a packageRule to tell Embroider how to disambiguate the case: | ||
|
||
```js | ||
// ember-cli-build.js | ||
compatBuild(app, Webpack, { | ||
packageRules: [ | ||
{ | ||
package: 'some-addon', | ||
semverRange: '<= 5.0', | ||
addonTemplates: { | ||
'templates/components/some-component.hbs': { | ||
disambiguate: { | ||
// or "helper" or "data" | ||
'thing': 'component', | ||
}, | ||
}, | ||
} | ||
} | ||
] | ||
}) | ||
``` | ||
|
||
2. `unsupported ambiguity between helper and component`: If you have forms that are ambiguous between being a helper and being a component, your settings for `staticComponents` and `staticHelpers` must now agree. | ||
|
||
For example, `{{some-thing value=1}}` could be a component or a helper. If you have code like this, Embroider options like `{ staticHelpers: true, staticComponents: false }` are no longer allowed. They need to both be true or both be false. | ||
|
||
You can fix this problem by either changing the forms to be unambiguous (in the same way as described in the previous section), or by changing one of the `staticCompnents` or `staticHelpers` flags to match the other one. | ||
|
||
|
||
3. The `safeToIgnore` packageRule used to only suppress an error when a component was not found. Now it prevents us from even trying to resolve the component. | ||
|
||
4. We no longer include a workaround for https://github.com/emberjs/ember.js/issues/19877, because Ember backported fixes to every major version that we support. Make sure you're on a supported Ember patch release. The oldest patch we support is ember-source 3.28.11. | ||
|
||
This particular bug is important to us because whenever `staticHelpers` is enabled, we convert all helper invocations to lexically scoped helper invocations. So if you have any class-based helpers and you have `staticHelpers` enabled you definitely need the Ember bugfix. | ||
|
||
|
||
|
||
|
||
|
||
|
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure how changeset is going to handle these package-less entries 😅
we'll see when #1404 updates!