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

Remove ability to select AVA 5 watcher #3257

Merged
merged 1 commit into from
Nov 7, 2023
Merged

Conversation

novemberborn
Copy link
Member

When I first retained the AVA 5 logic, it was under the assumption AVA 6 would be released in the summer with another breaking release later in the year. It's now autumn, Node.js 16 is end of life, and for now it's just not worth having this logic around.

Depending on how well the watcher works I'm not opposed to having it as an experimental "select your watch mode" feature though.

@novemberborn novemberborn changed the title Remove selection of AVA 5 watcher Remove ability to select AVA 5 watcher Nov 6, 2023
Base automatically changed from deps to main November 6, 2023 20:46
When I first retained the AVA 5 logic, it was under the assumption AVA 6 would be released in the summer with another breaking release later in the year. It's now autumn, Node.js 16 is end of life, and for now it's just not worth having this logic around.

Depending on how well the watcher works I'm not opposed to having it as an experimental "select your watch mode" feature though.
@novemberborn novemberborn merged commit e07179b into main Nov 7, 2023
15 checks passed
@novemberborn novemberborn deleted the remove-ava5-watcher branch November 7, 2023 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant