Skip to content

Updated typescript-eslint to v6 #54693

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

Merged
jakebailey merged 13 commits into from
Jul 14, 2023
Merged

Updated typescript-eslint to v6 #54693

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
a6f12dd
Updated typescript-eslint to v6 beta
JoshuaKGoldberg Jun 18, 2023
b001345
Corrected non-null chains
JoshuaKGoldberg Jun 18, 2023
4951fc2
Removed unnecessary rule meta.docs.recommended properties
JoshuaKGoldberg Jun 18, 2023
f810a22
Fix build complaint for argument-triva.cjs
JoshuaKGoldberg Jun 18, 2023
70caa0c
Merge branch 'main'
JoshuaKGoldberg Jun 22, 2023
052f6ed
Updated messaging (comment) for todo rules
JoshuaKGoldberg Jun 26, 2023
ee035f7
Merge branch 'main'
JoshuaKGoldberg Jul 6, 2023
0fde8d4
Bumped to latest alpha
JoshuaKGoldberg Jul 6, 2023
170c4e8
Merge branch 'main' into typescript-eslint-v6
JoshuaKGoldberg Jul 11, 2023
2b56c5a
typescript-eslint@v6 stable 🥳
JoshuaKGoldberg Jul 11, 2023
880768e
Review feedback: ^versions, eslint:recommended, config phrasing
JoshuaKGoldberg Jul 14, 2023
53e2ae1
A couple more disables
JoshuaKGoldberg Jul 14, 2023
0b72601
Remove last eslint rules in their recommended set
jakebailey Jul 14, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Removed unnecessary rule meta.docs.recommended properties
  • Loading branch information
@JoshuaKGoldberg
JoshuaKGoldberg committed Jun 18, 2023
commit 4951fc273b0b16bb7f0cfc6c7352f945e2f602a0
1 change: 0 additions & 1 deletion scripts/eslint/rules/argument-trivia.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ module.exports = createRule({
meta: {
docs: {
description: ``,
recommended: "error",
},
messages: {
argumentTriviaArgumentError: `Tag argument with parameter name`,
Expand Down
1 change: 0 additions & 1 deletion scripts/eslint/rules/debug-assert.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ module.exports = createRule({
meta: {
docs: {
description: ``,
recommended: "error",
},
messages: {
secondArgumentDebugAssertError: `Second argument to 'Debug.assert' should be a string literal`,
Expand Down
1 change: 0 additions & 1 deletion scripts/eslint/rules/jsdoc-format.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ module.exports = createRule({
meta: {
docs: {
description: ``,
recommended: "error",
},
messages: {
internalCommentInNonJSDocError: `@internal should not appear in non-JSDoc comment for declaration.`,
Expand Down
1 change: 0 additions & 1 deletion scripts/eslint/rules/no-double-space.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ module.exports = createRule({
meta: {
docs: {
description: ``,
recommended: "error",
},
messages: {
noDoubleSpaceError: `Use only one space`,
Expand Down
1 change: 0 additions & 1 deletion scripts/eslint/rules/no-in-operator.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ module.exports = createRule({
meta: {
docs: {
description: ``,
recommended: "error",
},
messages: {
noInOperatorError: `Don't use the 'in' keyword - use 'hasProperty' to check for key presence instead`,
Expand Down
1 change: 0 additions & 1 deletion scripts/eslint/rules/no-keywords.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ module.exports = createRule({
meta: {
docs: {
description: `disallows the use of certain TypeScript keywords as variable or parameter names`,
recommended: "error",
},
messages: {
noKeywordsError: `{{ name }} clashes with keyword/type`,
Expand Down
1 change: 0 additions & 1 deletion scripts/eslint/rules/no-type-assertion-whitespace.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ module.exports = createRule({
meta: {
docs: {
description: ``,
recommended: "error",
},
messages: {
noTypeAssertionWhitespace: `Excess trailing whitespace found around type assertion`,
Expand Down
1 change: 0 additions & 1 deletion scripts/eslint/rules/object-literal-surrounding-space.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ module.exports = createRule({
meta: {
docs: {
description: ``,
recommended: "error",
},
messages: {
leadingExcessStringError: `No trailing whitespace found on single-line object literal`,
Expand Down
1 change: 0 additions & 1 deletion scripts/eslint/rules/only-arrow-functions.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ module.exports = createRule({
meta: {
docs: {
description: `Disallows traditional (non-arrow) function expressions.`,
recommended: "error",
},
messages: {
onlyArrowFunctionsError: "non-arrow functions are forbidden",
Expand Down
1 change: 0 additions & 1 deletion scripts/eslint/rules/simple-indent.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ module.exports = createRule({
meta: {
docs: {
description: "Enforce consistent indentation",
recommended: "error",
},
messages: {
simpleIndentError: "4 space indentation expected",
Expand Down
1 change: 0 additions & 1 deletion scripts/eslint/rules/type-operator-spacing.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ module.exports = createRule({
meta: {
docs: {
description: ``,
recommended: "error",
},
messages: {
typeOperatorSpacingError: `The '|' and '&' operators must be surrounded by spaces`,
Expand Down