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

Support @ember/test-waiters v4 #9619

Merged
merged 1 commit into from
Dec 17, 2024

Conversation

NullVoxPopuli
Copy link
Contributor

@NullVoxPopuli NullVoxPopuli commented Dec 15, 2024

Description

v4 is a v2 addon

Notes for the release

Widen support of @ember/test-waiters to include v3 and v4

@runspired
Copy link
Contributor

this should replace #9614 but iirc there should be a lockfile change here too?

@runspired runspired added 🎯 canary PR is targeting canary (default) 🏷️ feat This PR introduces a new feature labels Dec 16, 2024
@NullVoxPopuli
Copy link
Contributor Author

I ran pnpm locally, and it did not generate one -- I suspect because y || x still satisfies y? 🤷

@runspired runspired merged commit 5dc213f into emberjs:main Dec 17, 2024
26 of 28 checks passed
runspired pushed a commit that referenced this pull request Dec 19, 2024
mansona pushed a commit to mansona/data that referenced this pull request Jan 21, 2025
# Conflicts:
#	packages/-ember-data/package.json
#	packages/ember/package.json
#	packages/legacy-compat/package.json
#	packages/request/package.json
#	pnpm-lock.yaml
#	tests/ember-data__adapter/package.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎯 canary PR is targeting canary (default) 🏷️ feat This PR introduces a new feature
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants