Skip to content

add tooling notes 17th June #631

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
59 changes: 59 additions & 0 deletions st-embroider/2024/2025-06-17.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,59 @@
# 2025-06-17

Note Taker: Chris Manson
Time Keeper: Chris Manson

## Attendees

Add yourself to the list if you attend and check the box!

- [x] Ed Faulkner (EF)
- [x] Chris Manson (CM)
- [x] Alex (A)
- [ ] Preston Sego (PS)
- [x] Simon Ihmig (SI)
- [x] Katie Gengler (KG)
- [ ] Krystan HuffMenne (KH)
- [ ] Peter Wagenet (PW)
- [ ] James Davis (JD)
- [x] TommyJR
-

## Topics

### sync on the self-imports in v2 addons situation <@NullVoxPopuli>

- no show might come back

### Prevent excessive recursion of findActiveAddons <@NullVoxPopuli, Simon>

- https://github.com/embroider-build/embroider/pull/2477
- currently blocked on stable CI being broken
- CM: we could pin macros on CI for now
- EF: good idea for now

### https://github.com/embroider-build/ember-auto-import/pull/659 <@NullVoxPopuli>

- no show
- not sure

### github-changelog structural changes: get from the latest tag on the current branch, and ignore pre-releases <@mansona>

- https://github.com/embroider-build/github-changelog/pull/52
- EF: seems good
- CM: should I make the "skip prerelase versions" an option
-

### updating the @ember/app-blueprint as part of the release process <@mansona>

- (large discussion about upgrade experience)
- probably will need to maintain a classic app blueprint even after the vite one is default
- we need to make sure that ember-cli-update doesn't try to upgrade to vite
- we could deprecate this build some time early in the 7.x series

<5 mins> can we stop releasing unstable now? Pollution of the release list with very little benefit: https://www.npmjs.com/package/@embroider/addon-dev?activeTab=versions <@mansona>
<2min> look at https://astexplorer.net/#/gist/2e263500cf1707871d072adbb3d5b00b/fa99860456ff82a762ec20d4233a5f4656e04759
Notice that it doesn't throw any error….
And the fix is https://github.com/tildeio/simple-html-tokenizer/pull/114 <@patricklx>
<10 mins> deep dive into the output of the new app blueprint https://github.com/ember-cli/ember-app-blueprint <@mansona>