Skip to content

Change require-expect rule default option to never-except-zero #375

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 27, 2023
Merged

Change require-expect rule default option to never-except-zero #375

merged 1 commit into from
Jun 27, 2023

Conversation

bmish
Copy link
Member

@bmish bmish commented Jun 27, 2023

Discussion here: #222 (comment)

Part of #222.

@bmish bmish added the breaking label Jun 27, 2023
@bmish bmish mentioned this pull request Jun 27, 2023
Closed
@coveralls
Copy link

Coverage Status

coverage: 100.0%. remained the same when pulling 79fc0bc on bmish:require-expect-never-except-zero-default into b68511a on platinumazure:master.

@platinumazure platinumazure merged commit 1c5935b into qunitjs:master Jun 27, 2023
@bmish bmish mentioned this pull request Jun 28, 2023
Merged
bmish added a commit to bmish/eslint-plugin-qunit that referenced this pull request Jun 28, 2023
@bmish
Merge branch 'master' into eslint-remote-tester
a29b470 
* master: (31 commits)
  chore: use ecmaVersion latest for eslint (qunitjs#379)
  chore: use node protocol for imports (qunitjs#378)
  upgrade: Upgrade dependencies in preparation for major release (qunitjs#377)
  feat!: Require eslint@^8.38, drop support for [email protected] (qunitjs#376)
  feat!: change require-expect rule default option to never-except-zero (qunitjs#375)
  feat!: strictly define node API (qunitjs#374)
  feat!: drop support for Node 12, 14, 17, 19 (qunitjs#373)
  Upgrade: Bump semver from 7.5.0 to 7.5.3 (qunitjs#372)
  Upgrade: Bump @typescript-eslint/parser from 5.59.0 to 5.60.0 (qunitjs#371)
  Upgrade: Bump release-it from 15.10.1 to 15.11.0 (qunitjs#366)
  Upgrade: Bump markdownlint-cli from 0.33.0 to 0.35.0 (qunitjs#370)
  Upgrade: Bump all-contributors-cli from 6.24.0 to 6.26.0 (qunitjs#359)
  Upgrade: Bump eslint-plugin-eslint-plugin from 5.0.8 to 5.1.0 (qunitjs#357)
  Upgrade: Bump vm2 from 3.9.17 to 3.9.19 (qunitjs#355)
  Upgrade: Bump eslint from 8.38.0 to 8.39.0 (qunitjs#342)
  Upgrade: Bump vm2 from 3.9.16 to 3.9.17 (qunitjs#341)
  Upgrade: Bump @typescript-eslint/parser from 5.54.1 to 5.59.0 (qunitjs#340)
  Upgrade: Bump eslint from 8.36.0 to 8.38.0 (qunitjs#335)
  Upgrade: Bump semver from 7.3.8 to 7.5.0 (qunitjs#339)
  Upgrade: Bump vm2 from 3.9.11 to 3.9.16 (qunitjs#336)
  ...
bmish added a commit to bmish/eslint-plugin-qunit that referenced this pull request Jun 28, 2023
@bmish
Merge branch 'master' into eslint-remote-tester
70a6101 
* master: (31 commits)
  chore: use ecmaVersion latest for eslint (qunitjs#379)
  chore: use node protocol for imports (qunitjs#378)
  upgrade: Upgrade dependencies in preparation for major release (qunitjs#377)
  feat!: Require eslint@^8.38, drop support for [email protected] (qunitjs#376)
  feat!: change require-expect rule default option to never-except-zero (qunitjs#375)
  feat!: strictly define node API (qunitjs#374)
  feat!: drop support for Node 12, 14, 17, 19 (qunitjs#373)
  Upgrade: Bump semver from 7.5.0 to 7.5.3 (qunitjs#372)
  Upgrade: Bump @typescript-eslint/parser from 5.59.0 to 5.60.0 (qunitjs#371)
  Upgrade: Bump release-it from 15.10.1 to 15.11.0 (qunitjs#366)
  Upgrade: Bump markdownlint-cli from 0.33.0 to 0.35.0 (qunitjs#370)
  Upgrade: Bump all-contributors-cli from 6.24.0 to 6.26.0 (qunitjs#359)
  Upgrade: Bump eslint-plugin-eslint-plugin from 5.0.8 to 5.1.0 (qunitjs#357)
  Upgrade: Bump vm2 from 3.9.17 to 3.9.19 (qunitjs#355)
  Upgrade: Bump eslint from 8.38.0 to 8.39.0 (qunitjs#342)
  Upgrade: Bump vm2 from 3.9.16 to 3.9.17 (qunitjs#341)
  Upgrade: Bump @typescript-eslint/parser from 5.54.1 to 5.59.0 (qunitjs#340)
  Upgrade: Bump eslint from 8.36.0 to 8.38.0 (qunitjs#335)
  Upgrade: Bump semver from 7.3.8 to 7.5.0 (qunitjs#339)
  Upgrade: Bump vm2 from 3.9.11 to 3.9.16 (qunitjs#336)
  ...
bmish added a commit to bmish/eslint-plugin-qunit that referenced this pull request Jun 28, 2023
@bmish
Merge branch 'master' into eslint-remote-tester
87c5b15 
* master: (31 commits)
  chore: use ecmaVersion latest for eslint (qunitjs#379)
  chore: use node protocol for imports (qunitjs#378)
  upgrade: Upgrade dependencies in preparation for major release (qunitjs#377)
  feat!: Require eslint@^8.38, drop support for [email protected] (qunitjs#376)
  feat!: change require-expect rule default option to never-except-zero (qunitjs#375)
  feat!: strictly define node API (qunitjs#374)
  feat!: drop support for Node 12, 14, 17, 19 (qunitjs#373)
  Upgrade: Bump semver from 7.5.0 to 7.5.3 (qunitjs#372)
  Upgrade: Bump @typescript-eslint/parser from 5.59.0 to 5.60.0 (qunitjs#371)
  Upgrade: Bump release-it from 15.10.1 to 15.11.0 (qunitjs#366)
  Upgrade: Bump markdownlint-cli from 0.33.0 to 0.35.0 (qunitjs#370)
  Upgrade: Bump all-contributors-cli from 6.24.0 to 6.26.0 (qunitjs#359)
  Upgrade: Bump eslint-plugin-eslint-plugin from 5.0.8 to 5.1.0 (qunitjs#357)
  Upgrade: Bump vm2 from 3.9.17 to 3.9.19 (qunitjs#355)
  Upgrade: Bump eslint from 8.38.0 to 8.39.0 (qunitjs#342)
  Upgrade: Bump vm2 from 3.9.16 to 3.9.17 (qunitjs#341)
  Upgrade: Bump @typescript-eslint/parser from 5.54.1 to 5.59.0 (qunitjs#340)
  Upgrade: Bump eslint from 8.36.0 to 8.38.0 (qunitjs#335)
  Upgrade: Bump semver from 7.3.8 to 7.5.0 (qunitjs#339)
  Upgrade: Bump vm2 from 3.9.11 to 3.9.16 (qunitjs#336)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@platinumazure platinumazure platinumazure approved these changes

Assignees
No one assigned
Labels
breaking
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bmish @coveralls @platinumazure