|
| 1 | +# 2025-11-25 |
| 2 | + |
| 3 | +Note Taker: Chris Manson |
| 4 | +Time Keeper: Chris Manson |
| 5 | + |
| 6 | +## Attendees |
| 7 | + |
| 8 | +Add yourself to the list if you attend and check the box! |
| 9 | + |
| 10 | +- [x] Ed Faulkner (EF) |
| 11 | +- [x] Chris Manson (CM) |
| 12 | +- [x] Alex (A) |
| 13 | +- [x] Preston Sego (PS) |
| 14 | +- [ ] Simon Ihmig (SI) |
| 15 | +- [x] Katie Gengler (KG) |
| 16 | +- [ ] Krystan HuffMenne (KH) |
| 17 | +- [ ] Peter Wagenet (PW) |
| 18 | +- [ ] James Davis (JD) |
| 19 | +- [ ] Liam Potter (LP) |
| 20 | +- [x] David Taylor |
| 21 | + |
| 22 | +## Topics |
| 23 | + |
| 24 | +### Review unreviewed PRs |
| 25 | + |
| 26 | +#### Use replacers for eslint and babel to fix the full path display during project generation |
| 27 | + |
| 28 | +- https://github.com/ember-cli/ember-app-blueprint/pull/149 |
| 29 | + |
| 30 | +- (discussion) generally a good change |
| 31 | +- CM: what's the value in adding this complexity to have it generate an app with newer node APIs |
| 32 | +- (long discussion) |
| 33 | +- (general agreement) we will remove that odd change and we can get this PR approved |
| 34 | + |
| 35 | +### amd bundle deprecation, release issues <ef4> |
| 36 | + |
| 37 | +- (long discussion about the merits and the way forward) |
| 38 | +- (discussion of the required updates in people's apps) |
| 39 | +- (general consensus that the trade offs are worth it and we can proceed) |
| 40 | +- (general discussion) optional-features needs to be rearchitected so we can use it more effectively |
| 41 | + - it really should just be part of ember-source |
| 42 | + |
| 43 | +### review "Embroider router support for ember source less than 4.12" <@void-mAlex> |
| 44 | + |
| 45 | +- https://github.com/embroider-build/embroider/pull/2624 |
| 46 | +- (general agreement) it's a good change |
| 47 | +- (general discussion) so we don't have tests for embroider router in the embroider monorepo |
| 48 | +- A: ember-engines has been a good test case for this and a few other things |
| 49 | +- |
| 50 | + |
| 51 | +### first investigations into optional babel plugin <@mansona> |
| 52 | + |
| 53 | +- (long discussion) |
| 54 | +- CM: so we can only do macros and loose mode templates |
| 55 | +- EF: all the logic of which one is loose and strict is defined in ember-babel-template-complication |
| 56 | +- ultimately we want to implement these in |
| 57 | + |
| 58 | +### bug with app files causing cycles in typescript <@mansona> |
| 59 | + |
| 60 | +- CM: should we probably just add app.ts to the list of ignored files? |
| 61 | +- EF: we need to ignore all "resolvable extensions" of app - this is defined somewhere |
| 62 | +- CM: I'll take a look at this |
| 63 | + |
| 64 | + |
0 commit comments