From 11902a266aa9ca677a64c845d57d92714fbbe4ee Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Sat, 3 Oct 2026 19:21:02 +0000 Subject: [PATCH] Version Packages --- .changeset/skipped-tests-ast-detection.md | 11 ----------- packages/eslint-plugin-code-quality/CHANGELOG.md | 12 ++++++++++++ packages/eslint-plugin-code-quality/package.json | 2 +- packages/eslint-plugin-code-quality/src/index.ts | 2 +- 4 files changed, 14 insertions(+), 13 deletions(-) delete mode 100644 .changeset/skipped-tests-ast-detection.md diff --git a/.changeset/skipped-tests-ast-detection.md b/.changeset/skipped-tests-ast-detection.md deleted file mode 100644 index ce4a625..0000000 --- a/.changeset/skipped-tests-ast-detection.md +++ /dev/null @@ -1,11 +0,0 @@ ---- -'@noctcore/eslint-plugin-code-quality': patch ---- - -`skipped-tests-need-tracking` no longer reports the text of a skip that is not a call. - -**Reports less.** The rule found `it.skip(`, `test.fixme(`, `xit(`, `xdescribe(` and `xtest(` by scanning each source line, so the same text inside a string, a template literal or a comment was reported as a skipped test. It now reads the skip from the call itself. A test file that lints probe code held as a string (a config wiring test with seeded violations, a `RuleTester` suite for a wrapper rule) can spell a skipped test in its own source, and a comment can mention `it.skip(` in prose. - -Every real skip that was reported before is still reported: the `.skip` / `.fixme` modifier on `it`, `test` or `describe` (also reached through a member, `test.describe.skip(`), and the `xit` / `xdescribe` / `xtest` aliases. The tracking marker is still looked up in the source text of the lookback window, so a marker in a comment counts as before. - -The report now sits on the callee (`it.skip`, `xit`) or on the `node:test` option, where it used to cover the whole line from column 1. The line is unchanged, so an `eslint-disable-next-line` keeps working. diff --git a/packages/eslint-plugin-code-quality/CHANGELOG.md b/packages/eslint-plugin-code-quality/CHANGELOG.md index f3335b2..ab6057d 100644 --- a/packages/eslint-plugin-code-quality/CHANGELOG.md +++ b/packages/eslint-plugin-code-quality/CHANGELOG.md @@ -1,5 +1,17 @@ # @noctcore/eslint-plugin-code-quality +## 0.4.2 + +### Patch Changes + +- [#64](https://github.com/noctcore/eslint-plugins/pull/64) [`aa1ae71`](https://github.com/noctcore/eslint-plugins/commit/aa1ae71b9db4eea014323e83bb615eeab04a201d) Thanks [@Shironex](https://github.com/Shironex)! - `skipped-tests-need-tracking` no longer reports the text of a skip that is not a call. + + **Reports less.** The rule found `it.skip(`, `test.fixme(`, `xit(`, `xdescribe(` and `xtest(` by scanning each source line, so the same text inside a string, a template literal or a comment was reported as a skipped test. It now reads the skip from the call itself. A test file that lints probe code held as a string (a config wiring test with seeded violations, a `RuleTester` suite for a wrapper rule) can spell a skipped test in its own source, and a comment can mention `it.skip(` in prose. + + Every real skip that was reported before is still reported: the `.skip` / `.fixme` modifier on `it`, `test` or `describe` (also reached through a member, `test.describe.skip(`), and the `xit` / `xdescribe` / `xtest` aliases. The tracking marker is still looked up in the source text of the lookback window, so a marker in a comment counts as before. + + The report now sits on the callee (`it.skip`, `xit`) or on the `node:test` option, where it used to cover the whole line from column 1. The line is unchanged, so an `eslint-disable-next-line` keeps working. + ## 0.4.1 ### Patch Changes diff --git a/packages/eslint-plugin-code-quality/package.json b/packages/eslint-plugin-code-quality/package.json index 5996c4f..43359ba 100644 --- a/packages/eslint-plugin-code-quality/package.json +++ b/packages/eslint-plugin-code-quality/package.json @@ -1,6 +1,6 @@ { "name": "@noctcore/eslint-plugin-code-quality", - "version": "0.4.1", + "version": "0.4.2", "description": "Portable code-quality, comment-hygiene and test-discipline ESLint rules: guard clauses, deterministic time, no stray process.exit, and tests that cannot pass vacuously.", "license": "MIT", "type": "module", diff --git a/packages/eslint-plugin-code-quality/src/index.ts b/packages/eslint-plugin-code-quality/src/index.ts index 139ecc1..14d5c6e 100644 --- a/packages/eslint-plugin-code-quality/src/index.ts +++ b/packages/eslint-plugin-code-quality/src/index.ts @@ -3,7 +3,7 @@ import { rules } from './rules'; /** Flat-config namespace: rule ids are keyed `noctcore-code-quality/`. */ const NAMESPACE = 'noctcore-code-quality'; -const VERSION = '0.4.1'; +const VERSION = '0.4.2'; const plugin = { meta: { name: '@noctcore/eslint-plugin-code-quality', version: VERSION },