-
Notifications
You must be signed in to change notification settings - Fork 58
Prepare Release #833
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
Prepare Release #833
Conversation
1afbaab to
3dbadc5
Compare
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.
[NOT READY FOR MERGE]
(as of 2025-03-23)
We don't want @glint/template to release as a major version bump.
I don't think any meaningful changes have actually happened within it, but it probably got caught up in wide-sweeping changes that are breaking in the same PR.
When we're ready to release, we'll need to hand-edit the .release-plan.json and @glint/template/package.json to change the version bump to be a minor release.
|
oh actually, it is breaking. because we dropped support for GlimmerX and formal support for DT Types. HMMM. projects that already include |
|
Also, ts-plugin-test-app needs to be private |
3dbadc5 to
6c240bd
Compare
|
fixed @patricklx 🎉 |
76a7b0a to
4008d7a
Compare
dd8f645 to
1a655bc
Compare
f62ea84 to
40cb70d
Compare
7b64361 to
891a959
Compare
891a959 to
fd4daa4
Compare
fd4daa4 to
12df346
Compare
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.
Re: declared "breaking" changes in @glint/template
- only comments: https://github.com/typed-ember/glint/pull/716/files#diff-2da8504290f080a9d0d2ab0820be3a8d4ff6dbce684b3ae65ca216d8d85c7f15L30
- just removed a
dependenciesentry on a@types: https://github.com/typed-ember/glint/pull/820/files#diff-db9c93b67e37a1ca885c6ecaba477c13d3a487a4dabab5a740bb6212def2554dL21- this isn't breaking because our whole ecosystem relies on peer-provided types.
So, I'm going to manually tell @glint/template to be a patch release, rather than a major (it's declared as a major due to basically re-writing Glint, which resulted in some sloppy git practices).
However!!!!
Here is a test of @glint/template@unstable (tracking every main commit) against a project that was previously using @glint/template@^1.5: universal-ember/ember-primitives#491
It's currently failing. Which means "something" did break something. We need to fix this, because we don't want to release a major of @glint/template, ideally -- since many packages depend on it -- we don't want to force a release that expands their version range to || ^2.0.0-alpha.0 if we don't have to.
In particular, it looks like our element subclassing is failing?
src/components/keys.gts:34:51 - error TS2345: Argument of type 'HTMLSpanElement' is not assignable to parameter of type 'HTMLElement'.
Types of property '[GlintSymbol]' are incompatible.
Type '"HTMLSpanElement"' is not assignable to type '"HTMLElement"'.
34 <span class="ember-primitives__key-combination" ...attributes>
and
src/components/menu.gts:175:7 - error TS2345: Argument of type 'HTMLDivElement' is not assignable to parameter of type 'HTMLElement'.
Types of property '[GlintSymbol]' are incompatible.
Type '"HTMLDivElement"' is not assignable to type '"HTMLElement"'.
175 {{installContent isOpen=@isOpen triggerElement=@triggerElement}}
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
where this modifier has specified HTMLElement, and the element that the modifier is on is a HTMLDivElement. This should be allowed because HTMLElement is saying "I take any HTML".
|
shouldn't |
|
Template has a bug which is blocking any releasing right now See: #903 There is some discussion in discord about this, too, iirc |
3e0d833 to
d9e0fe5
Compare
d9e0fe5 to
b491f6b
Compare
|
Thanks to @mansona, our alpha versions look good. All publishable packages will be alpha as of the merge of this PR. The goal:
|
This PR is a preview of the release that release-plan has prepared. To release you should just merge this PR 👍
Release (2025-06-16)
💥 Breaking Change
@glint/core,@glint/environment-ember-loose,@glint/environment-ember-template-imports,@glint/template@glint/core,@glint/tsserver-plugin@glint/coreglintcli build tests, parity withvue-tsc(@machty)@glint/environment-ember-loose@glint/core,@glint/environment-ember-loose,@glint/environment-ember-template-imports@glint/core,@glint/template🚀 Enhancement
@glint/core,@glint/tsserver-plugin<template>portions of .gts files (@machty)@glint/corecomponents/folder (@machty)@glint/core,@glint/template,@glint/tsserver-plugin@glint/core,@glint/environment-ember-loose,@glint/environment-ember-template-imports,@glint/template@glint/core,@glint/environment-ember-loose,@glint/environment-ember-template-imports🐛 Bug Fix
@glint/template@glint/core,@glint/tsserver-plugin@glint/core.gts/.gjsextensions (@machty)@glint/environment-ember-template-importssatisfieson components. (@NullVoxPopuli)@glint/core,@glint/environment-ember-template-imports@glint/tsserver-plugin@glint/environment-ember-looseeach-in(@mogstad)📝 Documentation
export defaultfrom the Template-only components example (@Windvis)@builtin typescriptextension forV2 (@johanrd)@glint/type-test🏠 Internal
@glint/template@glint/templatethat cause dependency problems to their own package (@NullVoxPopuli)@glint/core,@glint/environment-ember-loose,@glint/environment-ember-template-imports,@glint/tsserver-plugin,@glint/type-test@glint/environment-ember-template-imports@glint/environment-ember-loose,@glint/environment-ember-template-imports,@glint/type-test@glint/core@glint/core,@glint/environment-ember-loose,@glint/environment-ember-template-imports,@glint/template,@glint/type-test@glint/core,@glint/environment-ember-looseCommitters: 12