Skip to content

Conversation

skovy
Copy link
Collaborator

@skovy skovy commented Sep 17, 2022

BREAKING CHANGE: remove prefer-wait-for rule since the deprecated helpers were removed

Checks

  • I have read the contributing guidelines.
  • If some rule is added/updated/removed, I've regenerated the rules list (npm run generate:rules-list)
  • If some rule meta info is changed, I've regenerated the plugin shared configs (npm run generate:configs)

Changes

Not sure if there is a v6 or alpha branch?

Context

Resolves #627

BREAKING CHANGE: remove prefer-wait-for rule
@Belco90
Copy link
Member

Belco90 commented Sep 18, 2022

Hey @skovy, thanks for your contribution!

I'm going to create v6 so we can target this PR to that branch.

@MichaelDeBoey
Copy link
Member

@Belco90 alpha should be created instead so we can download latest changes from npm to test them out locally in our own projects

@MichaelDeBoey MichaelDeBoey changed the base branch from main to alpha September 19, 2022 10:15
@MichaelDeBoey MichaelDeBoey changed the title chore: remove prefer-wait-for chore: remove prefer-wait-for rule Sep 19, 2022
@MichaelDeBoey MichaelDeBoey changed the title chore: remove prefer-wait-for rule feat: remove prefer-wait-for rule Sep 19, 2022
@Belco90
Copy link
Member

Belco90 commented Sep 19, 2022

@Belco90 alpha should be created instead so we can download latest changes from npm to test them out locally in our own projects

Even better, thanks for that!

@Belco90 Belco90 merged commit 5aa0194 into testing-library:alpha Sep 19, 2022
@github-actions
Copy link

🎉 This PR is included in version 6.0.0-alpha.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@skovy skovy deleted the pr/remove-prefer-wait-for branch September 19, 2022 15:38
@MichaelDeBoey MichaelDeBoey added the v6 Next major v6 label Sep 20, 2022
@MichaelDeBoey MichaelDeBoey added the BREAKING CHANGE This change will require a major version bump label Oct 4, 2022
MichaelDeBoey pushed a commit that referenced this pull request Oct 4, 2022
BREAKING CHANGE: `prefer-wait-for` is now removed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BREAKING CHANGE This change will require a major version bump released on @alpha v6 Next major v6
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Remove prefer-wait-for rule
3 participants