Skip to content

add notes for tooling team 5th and 12th August #636

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 1 commit into from
Aug 19, 2025
Merged
Show file tree
Hide file tree
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
7 changes: 3 additions & 4 deletions st-embroider/2024/2025-07-29.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,10 @@ Add yourself to the list if you attend and check the box!
- [ ] Krystan HuffMenne (KH)
- [ ] Peter Wagenet (PW)
- [ ] James Davis (JD)
- [ ] Evo
- [x] Evo
- [x] TommyJR
- [ ] Robbytx
- [ ] RKS
- [ ]
- [x] Robbytx
- [x] RKS

## Topics

Expand Down
33 changes: 33 additions & 0 deletions st-embroider/2024/2025-08-05.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,33 @@
# YYYY-MM-DD

Note Taker:
Time Keeper:

## Attendees

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

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

## Topics

not a lot of poeople today so we just focused on the PR for `@ember/debug`

### `@ember/debug` PR to embroider

- https://github.com/embroider-build/embroider/pull/2501
- review and build context to see if it's ok to merge
- looks good
- merged
- released

56 changes: 56 additions & 0 deletions st-embroider/2024/2025-08-12.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,56 @@
# 2025-08-12

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)
- [x] Preston Sego (PS)
- [ ] Simon Ihmig (SI)
- [x] Katie Gengler (KG)
- [ ] Krystan HuffMenne (KH)
- [x] Peter Wagenet (PW)
- [ ] James Davis (JD)
- [ ] Liam Potter (LP)
- [x] Cortrah
- [x] tommyjr

## Topics

No agenda today

### Looking into speed of building question

- CM: https://github.com/mainmatter/build-start-rebuild-perf
- we're looking into build speeds to keep them at least the same as before
- PS: I'm looking into improving number of reloads during depScan
- https://github.com/NullVoxPopuli/limber/pull/1968

### reviewing the embroider roadmap

- 2 topics that are ready for people to work on if anyone has the time
- deprecate non-serialisable babel plugins
- Deprecate app.import
- just writing the deprecation guides can make a massive difference

### discussion about slow tests

- A: 3x slowdown
- (long discussion about possible problems)
- A: I'll go and try some of the suggestions and report back

### Render assertion

- https://github.com/glimmerjs/glimmer-vm/issues/1762
- (long discussions)
- Main take-away:
- don't ever set tracked state during a render (we should bake the assertion more strict)
- you should never create tracked state from tracked state. You should refactor it to be a getter



1 change: 1 addition & 0 deletions st-embroider/template.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ Add yourself to the list if you attend and check the box!
- [ ] Krystan HuffMenne (KH)
- [ ] Peter Wagenet (PW)
- [ ] James Davis (JD)
- [ ] Liam Potter (LP)
- [ ] Your name here (INITIALS)

## Topics
Expand Down