Skip to content

Add new "noFocusedTest" rule #44

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

Merged

Conversation

elaichenkov
Copy link
Contributor

Description:

image

@elaichenkov elaichenkov requested a review from mskelton April 13, 2022 13:07
Copy link
Member

@mskelton mskelton left a comment

Choose a reason for hiding this comment

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

Nice work!

@mskelton mskelton merged commit 13afef3 into playwright-community:master Apr 13, 2022
elaichenkov added a commit to elaichenkov/eslint-plugin-playwright that referenced this pull request Apr 23, 2022
* rule: add new noElementHandle rule

* chore: move end range to const

* fix: getRange method for non-awaiting statements

* test: add additional tests for the noElementHandle rule

* chore: improve noElementHandle rule

* docs: update the wording of noElementHandle rule

* rule: update message to match with other rule naming conventions

* rule: change noElementHandle to warn level

* rule: noElementHandle change fixable to suggestion

* test: add valid tests for noElementHandle rule

* rule: change type for noElementHandle to suggestion

* rule: update noElementHandle rule's message for suggestions

* rule: update noElementHandle with getRange method

* feat: add new noFocusedTest rule

* chore: remove unneeded function in test. change DS for isTestGroup function

* chore: remove valid function
mskelton pushed a commit that referenced this pull request Apr 24, 2022
* rule: add new noElementHandle rule

* chore: move end range to const

* fix: getRange method for non-awaiting statements

* test: add additional tests for the noElementHandle rule

* chore: improve noElementHandle rule

* docs: update the wording of noElementHandle rule

* rule: update message to match with other rule naming conventions

* rule: change noElementHandle to warn level

* rule: noElementHandle change fixable to suggestion

* test: add valid tests for noElementHandle rule

* rule: change type for noElementHandle to suggestion

* rule: update noElementHandle rule's message for suggestions

* rule: update noElementHandle with getRange method

* devops: add NPM publish workflow (#42)

* Add no eval rule (#41)

* rule: add new noElementHandle rule

* chore: move end range to const

* fix: getRange method for non-awaiting statements

* test: add additional tests for the noElementHandle rule

* chore: improve noElementHandle rule

* docs: update the wording of noElementHandle rule

* rule: update message to match with other rule naming conventions

* rule: change noElementHandle to warn level

* rule: noElementHandle change fixable to suggestion

* test: add valid tests for noElementHandle rule

* rule: change type for noElementHandle to suggestion

* rule: update noElementHandle rule's message for suggestions

* rule: update noElementHandle with getRange method

* rule: add new noEval rule

* utils: add ast helpers

* chore: add newline at end of file

* chore: change double quote to single

* chore: add link for the noEval rule

* utils: add isCalleeProperty function

* Add new "noFocusedTest" rule (#44)

* rule: add new noElementHandle rule

* chore: move end range to const

* fix: getRange method for non-awaiting statements

* test: add additional tests for the noElementHandle rule

* chore: improve noElementHandle rule

* docs: update the wording of noElementHandle rule

* rule: update message to match with other rule naming conventions

* rule: change noElementHandle to warn level

* rule: noElementHandle change fixable to suggestion

* test: add valid tests for noElementHandle rule

* rule: change type for noElementHandle to suggestion

* rule: update noElementHandle rule's message for suggestions

* rule: update noElementHandle with getRange method

* feat: add new noFocusedTest rule

* chore: remove unneeded function in test. change DS for isTestGroup function

* chore: remove valid function

* Add new "noWaitForTimeout" rule (#46)

* rule: add new noElementHandle rule

* chore: move end range to const

* fix: getRange method for non-awaiting statements

* test: add additional tests for the noElementHandle rule

* chore: improve noElementHandle rule

* docs: update the wording of noElementHandle rule

* rule: update message to match with other rule naming conventions

* rule: change noElementHandle to warn level

* rule: noElementHandle change fixable to suggestion

* test: add valid tests for noElementHandle rule

* rule: change type for noElementHandle to suggestion

* rule: update noElementHandle rule's message for suggestions

* rule: update noElementHandle with getRange method

* feature: add noWaitForTimeout rule

* Add new "noSkippedTest" rule (#45)

* rule: add new noElementHandle rule

* chore: move end range to const

* fix: getRange method for non-awaiting statements

* test: add additional tests for the noElementHandle rule

* chore: improve noElementHandle rule

* docs: update the wording of noElementHandle rule

* rule: update message to match with other rule naming conventions

* rule: change noElementHandle to warn level

* rule: noElementHandle change fixable to suggestion

* test: add valid tests for noElementHandle rule

* rule: change type for noElementHandle to suggestion

* rule: update noElementHandle rule's message for suggestions

* rule: update noElementHandle with getRange method

* feat: add noSkippedTest rule

* chore: add missed import

* docs: add missed  in example

* chore: add 'only' annotation in valid test

* utils: add runRuleTest helper function

Co-authored-by: Max Schmitt <[email protected]>
@mxschmitt mxschmitt mentioned this pull request Apr 28, 2022
3 tasks
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.

2 participants