Skip to content

Commit 5c2f561

Browse files
committed
refactor(no-raw-special-elements): rename rule
1 parent a003664 commit 5c2f561

File tree

12 files changed

+23
-18
lines changed

12 files changed

+23
-18
lines changed

.changeset/thin-dryers-know.md

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'eslint-plugin-svelte': major
3+
---
4+
5+
Renamed `no-deprecated-raw-special-elements` in favor of `no-raw-special-elements`

README.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -320,7 +320,7 @@ These rules relate to possible syntax or logic errors in Svelte code:
320320
| Rule ID | Description | |
321321
|:--------|:------------|:---|
322322
| [svelte/infinite-reactive-loop](https://sveltejs.github.io/eslint-plugin-svelte/rules/infinite-reactive-loop/) | Svelte runtime prevents calling the same reactive statement twice in a microtask. But between different microtask, it doesn't prevent. | |
323-
| [svelte/no-deprecated-raw-special-elements](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-deprecated-raw-special-elements/) | Recommends not using raw special elements in Svelte versions previous to 5. | :wrench: |
323+
| [svelte/no-raw-special-elements](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-raw-special-elements/) | Recommends not using raw special elements in Svelte versions previous to 5. | :wrench: |
324324
| [svelte/no-dom-manipulating](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-dom-manipulating/) | disallow DOM manipulating | |
325325
| [svelte/no-dupe-else-if-blocks](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-dupe-else-if-blocks/) | disallow duplicate conditions in `{#if}` / `{:else if}` chains | :star: |
326326
| [svelte/no-dupe-on-directives](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-dupe-on-directives/) | disallow duplicate `on:` directives | |

docs/rules.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ These rules relate to possible syntax or logic errors in Svelte code:
1717
| Rule ID | Description | |
1818
| :------------------------------------------------------------------------------------------------------------- | :------------------------------------------------------------------------------------------------------------------------------------- | :------------- |
1919
| [svelte/infinite-reactive-loop](./rules/infinite-reactive-loop.md) | Svelte runtime prevents calling the same reactive statement twice in a microtask. But between different microtask, it doesn't prevent. | |
20-
| [svelte/no-deprecated-raw-special-elements](./rules/no-deprecated-raw-special-elements.md) | Recommends not using raw special elements in Svelte versions previous to 5. | :wrench: |
20+
| [svelte/no-raw-special-elements](./rules/no-raw-special-elements.md) | Recommends not using raw special elements in Svelte versions previous to 5. | :wrench: |
2121
| [svelte/no-dom-manipulating](./rules/no-dom-manipulating.md) | disallow DOM manipulating | |
2222
| [svelte/no-dupe-else-if-blocks](./rules/no-dupe-else-if-blocks.md) | disallow duplicate conditions in `{#if}` / `{:else if}` chains | :star: |
2323
| [svelte/no-dupe-on-directives](./rules/no-dupe-on-directives.md) | disallow duplicate `on:` directives | |

docs/rules/no-deprecated-raw-special-elements.md renamed to docs/rules/no-raw-special-elements.md

+5-5
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
---
22
pageClass: 'rule-details'
33
sidebarDepth: 0
4-
title: 'svelte/no-deprecated-raw-special-elements'
4+
title: 'svelte/no-raw-special-elements'
55
description: 'Recommends not using raw special elements in Svelte versions previous to 5.'
66
since: 'v3.0.0-next.1'
77
---
88

9-
# svelte/no-deprecated-raw-special-elements
9+
# svelte/no-raw-special-elements
1010

1111
> Recommends not using raw special elements in Svelte versions previous to 5.
1212
@@ -20,7 +20,7 @@ This rule reports the usage of `head`, `body`, `window`, `document`, `element` a
2020

2121
```svelte
2222
<script>
23-
/* eslint svelte/no-deprecated-raw-special-elements: "error" */
23+
/* eslint svelte/no-raw-special-elements: "error" */
2424
</script>
2525
2626
<!-- ✓ GOOD -->
@@ -48,5 +48,5 @@ This rule was introduced in eslint-plugin-svelte v3.0.0-next.1
4848

4949
## :mag: Implementation
5050

51-
- [Rule source](https://github.com/sveltejs/eslint-plugin-svelte/blob/main/packages/eslint-plugin-svelte/src/rules/no-deprecated-raw-special-elements.ts)
52-
- [Test source](https://github.com/sveltejs/eslint-plugin-svelte/blob/main/packages/eslint-plugin-svelte/tests/src/rules/no-deprecated-raw-special-elements.ts)
51+
- [Rule source](https://github.com/sveltejs/eslint-plugin-svelte/blob/main/packages/eslint-plugin-svelte/src/rules/no-raw-special-elements.ts)
52+
- [Test source](https://github.com/sveltejs/eslint-plugin-svelte/blob/main/packages/eslint-plugin-svelte/tests/src/rules/no-raw-special-elements.ts)

packages/eslint-plugin-svelte/src/rule-types.ts

+5-5
Original file line numberDiff line numberDiff line change
@@ -104,11 +104,6 @@ export interface RuleOptions {
104104
* @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-at-html-tags/
105105
*/
106106
'svelte/no-at-html-tags'?: Linter.RuleEntry<[]>
107-
/**
108-
* Recommends not using raw special elements in Svelte versions previous to 5.
109-
* @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-deprecated-raw-special-elements/
110-
*/
111-
'svelte/no-deprecated-raw-special-elements'?: Linter.RuleEntry<[]>
112107
/**
113108
* disallow DOM manipulating
114109
* @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-dom-manipulating/
@@ -195,6 +190,11 @@ export interface RuleOptions {
195190
* @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-object-in-text-mustaches/
196191
*/
197192
'svelte/no-object-in-text-mustaches'?: Linter.RuleEntry<[]>
193+
/**
194+
* Recommends not using raw special elements in Svelte versions previous to 5.
195+
* @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-raw-special-elements/
196+
*/
197+
'svelte/no-raw-special-elements'?: Linter.RuleEntry<[]>
198198
/**
199199
* it's not necessary to define functions in reactive statements
200200
* @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-reactive-functions/

packages/eslint-plugin-svelte/src/rules/no-deprecated-raw-special-elements.ts renamed to packages/eslint-plugin-svelte/src/rules/no-raw-special-elements.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import { createRule } from '../utils/index.js';
44
const INVALID_HTML_ELEMENTS = ['head', 'body', 'window', 'document', 'element', 'options'];
55
const VALID_PREFIX = 'svelte:';
66

7-
export default createRule('no-deprecated-raw-special-elements', {
7+
export default createRule('no-raw-special-elements', {
88
meta: {
99
docs: {
1010
description: 'Recommends not using raw special elements in Svelte versions previous to 5.',

packages/eslint-plugin-svelte/src/utils/rules.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ import maxAttributesPerLine from '../rules/max-attributes-per-line.js';
2020
import mustacheSpacing from '../rules/mustache-spacing.js';
2121
import noAtDebugTags from '../rules/no-at-debug-tags.js';
2222
import noAtHtmlTags from '../rules/no-at-html-tags.js';
23-
import noDeprecatedRawSpecialElements from '../rules/no-deprecated-raw-special-elements.js';
2423
import noDomManipulating from '../rules/no-dom-manipulating.js';
2524
import noDupeElseIfBlocks from '../rules/no-dupe-else-if-blocks.js';
2625
import noDupeOnDirectives from '../rules/no-dupe-on-directives.js';
@@ -38,6 +37,7 @@ import noInspect from '../rules/no-inspect.js';
3837
import noNavigationWithoutBase from '../rules/no-navigation-without-base.js';
3938
import noNotFunctionHandler from '../rules/no-not-function-handler.js';
4039
import noObjectInTextMustaches from '../rules/no-object-in-text-mustaches.js';
40+
import noRawSpecialElements from '../rules/no-raw-special-elements.js';
4141
import noReactiveFunctions from '../rules/no-reactive-functions.js';
4242
import noReactiveLiterals from '../rules/no-reactive-literals.js';
4343
import noReactiveReassign from '../rules/no-reactive-reassign.js';
@@ -91,7 +91,6 @@ export const rules = [
9191
mustacheSpacing,
9292
noAtDebugTags,
9393
noAtHtmlTags,
94-
noDeprecatedRawSpecialElements,
9594
noDomManipulating,
9695
noDupeElseIfBlocks,
9796
noDupeOnDirectives,
@@ -109,6 +108,7 @@ export const rules = [
109108
noNavigationWithoutBase,
110109
noNotFunctionHandler,
111110
noObjectInTextMustaches,
111+
noRawSpecialElements,
112112
noReactiveFunctions,
113113
noReactiveLiterals,
114114
noReactiveReassign,
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { RuleTester } from '../../utils/eslint-compat.js';
2-
import rule from '../../../src/rules/no-deprecated-raw-special-elements.js';
2+
import rule from '../../../src/rules/no-raw-special-elements.js';
33
import { loadTestCases } from '../../utils/utils.js';
44

55
const tester = new RuleTester({
@@ -10,7 +10,7 @@ const tester = new RuleTester({
1010
});
1111

1212
tester.run(
13-
'no-deprecated-raw-special-elements',
13+
'no-raw-special-elements',
1414
rule as any,
15-
loadTestCases('no-deprecated-raw-special-elements')
15+
loadTestCases('no-raw-special-elements')
1616
);

0 commit comments

Comments
 (0)