Skip to content

Conversation

Belco90
Copy link
Member

@Belco90 Belco90 commented Aug 5, 2023

Checks

Changes

This PR contains all the changes for v6.0.0. You can install it via the alpha tag before it gets released officially.

Context

n/a

skovy and others added 22 commits October 5, 2022 00:04
BREAKING CHANGE: `prefer-wait-for` is now removed
BREAKING CHANGE: `no-render-in-setup` is now called `no-render-in-lifecycle`
BREAKING CHANGE: `no-manual-cleanup` is now enabled by default in the React & Vue configs
…660)

BREAKING CHANGE: `no-global-regexp-flag-in-query` is now enabled by default in all configs
…or `user-event` (#652)

BREAKING CHANGE: `await-fire-event` is now called `await-async-event`
BREAKING CHANGE: `no-node-access` is now enabled by default in the DOM config
…default (#663)

BREAKING CHANGE: `no-debugging-utils` now enables all debug methods in all configs by default
BREAKING CHANGE: `no-await-sync-query` is now called `no-await-sync-queries`
…fault (#667)

BREAKING CHANGE: `no-await-sync-events` is now enabled by default in the DOM, Angular & React configs
BREAKING CHANGE: `await-async-query` is now called `await-async-queries`
# Conflicts:
#	README.md
#	docs/rules/await-fire-event.md
#	docs/rules/no-render-in-lifecycle.md
#	docs/rules/prefer-wait-for.md
#	lib/rules/await-fire-event.ts
#	tests/__snapshots__/index.test.ts.snap
#	tests/index.test.ts
#	tests/lib/rules/await-fire-event.test.ts
# Conflicts:
#	README.md
#	lib/rules/no-debugging-utils.ts
#	tests/index.test.ts
@Belco90 Belco90 added BREAKING CHANGE This change will require a major version bump v6 Next major v6 labels Aug 5, 2023
@Belco90 Belco90 requested a review from a team August 5, 2023 11:54
@Belco90 Belco90 self-assigned this Aug 5, 2023
@Belco90
Copy link
Member Author

Belco90 commented Aug 5, 2023

I need to install the alpha version in some projects and give it a try.

@Belco90 Belco90 marked this pull request as draft August 5, 2023 12:02
@Belco90 Belco90 marked this pull request as ready for review August 7, 2023 08:04
@Belco90
Copy link
Member Author

Belco90 commented Aug 7, 2023

Tested by installing the alpha in other projects. All good!

Comment on lines +33 to +35
dom: ['error', { eventModule: 'userEvent' }],
angular: ['error', { eventModule: 'userEvent' }],
react: ['error', { eventModule: 'userEvent' }],
Copy link
Member

Choose a reason for hiding this comment

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

Should we use an array here as well for consistency?

Suggested change
dom: ['error', { eventModule: 'userEvent' }],
angular: ['error', { eventModule: 'userEvent' }],
react: ['error', { eventModule: 'userEvent' }],
dom: ['error', { eventModule: ['userEvent'] }],
angular: ['error', { eventModule: ['userEvent'] }],
react: ['error', { eventModule: ['userEvent'] }],

Copy link
Member Author

Choose a reason for hiding this comment

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

Both ways are accepted, and we are only setting one value, so it's fine.

@Belco90 Belco90 merged commit b0fa89d into main Aug 11, 2023
@github-actions
Copy link

🎉 This PR is included in version 6.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@Belco90 Belco90 deleted the alpha branch April 16, 2024 08:52
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 v6 Next major v6

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

3 participants