Skip to content
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

add notes for embroider 16th May #503

Merged
merged 1 commit into from
May 23, 2023
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
47 changes: 47 additions & 0 deletions st-embroider/2023/2023-05-16.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,47 @@
# 2023-05-16

Note Taker: Chris Manson
Time Keeper: Chris Manson

## Attendees

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

- [x] Ed Falkner (EF)
- [x] Chris Manson (CM)
- [x] Aaron Chambers (AC)
- [x] Alex (A)
- [x] Preston Sego (PS)
- [x] Dave Laird (DL)

## Topics

### ember 5 + CI, oh no! <@NullVoxPopuli>

- EF: we were debugging this in the extended office hours
- CM pointed out that we were pinning an old version of ember data
- we identified an issue with ember-test-waiters and we have a good compat adapter for it now

### addon-dev optimizations <@NullVoxPopuli>

- https://github.com/embroider-build/embroider/pull/1423
- PS: I wanted to find a way to test this
- EF: we do have a way to do rebuild tests with scenario tester
- `compat-dummy-app-test` does this. It uses a rebuilder tests infrastructure
- this is specifically for ember-cli (doing some broccoli specific things)
- looking at the PR again, we can probably just use JSON.stringify and compare the strings
- and that would mean we wouldn't need to add a test
- PS: I'll update the PR
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR updated


### release status? <@mansona>

- EF: I've updated the release infra
- before we had independent versioning we were able to use release it fine, it just can't help you plan version bumps
- I haven't opened a PR because I want to add some documentation first
- I haven't integrated it with infra yet
- we will likely need to intervene e.g. if you have a label on a PR that is breaking but it touches another package that isn't breaking
- (summary of some of the changes to peerDependencies and workspace dependencies)
- high-level overview is that changesets doesn't work with our needs
- too much work to make it work the way we need
- (demo of the new release infrastructure)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

related: embroider-build/embroider#1430 (changeset removal PR)

3 changes: 3 additions & 0 deletions st-embroider/template.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@ Add yourself to the list if you attend and check the box!

- [ ] Ed Falkner (EF)
- [ ] Chris Manson (CM)
- [ ] Aaron Chambers (AC)
- [ ] Alex (A)
- [ ] Preston Sego (PS)
- [ ] Your name here (INITIALS)

## Topics
Expand Down