Skip to content

chore: enable @typescript-eslint/no-unnecessary-template-expression#110219

Merged
JoshuaKGoldberg merged 2 commits into
masterfrom
tseslint-no-unnecessary-template-expression
Mar 11, 2026
Merged

chore: enable @typescript-eslint/no-unnecessary-template-expression#110219
JoshuaKGoldberg merged 2 commits into
masterfrom
tseslint-no-unnecessary-template-expression

Conversation

@JoshuaKGoldberg
Copy link
Copy Markdown
Member

@JoshuaKGoldberg JoshuaKGoldberg commented Mar 9, 2026

Follows up on #110153 (comment) by fully enabling @typescript-eslint/no-unnecessary-template-expression on the codebase.

tl;dr: the rule reports on template expressions which take in an already-string value, but don't add anything to it.

const original = 'apple';
const modified = `${apple}`; // 'apple'

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 9, 2026
@JoshuaKGoldberg JoshuaKGoldberg marked this pull request as ready for review March 9, 2026 17:13
@JoshuaKGoldberg JoshuaKGoldberg requested review from a team as code owners March 9, 2026 17:13
@JoshuaKGoldberg JoshuaKGoldberg force-pushed the tseslint-no-unnecessary-template-expression branch from bac5c08 to 3db5fbc Compare March 10, 2026 18:24
@JoshuaKGoldberg JoshuaKGoldberg requested a review from a team as a code owner March 10, 2026 18:24
@JoshuaKGoldberg JoshuaKGoldberg merged commit 87b1201 into master Mar 11, 2026
62 checks passed
@JoshuaKGoldberg JoshuaKGoldberg deleted the tseslint-no-unnecessary-template-expression branch March 11, 2026 12:38
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants