Skip to content

Extract rule: template-deprecated-inline-view-helper#2450

Merged
NullVoxPopuli merged 1 commit intoember-cli:masterfrom
NullVoxPopuli:nvp/template-lint-extract-rule-template-deprecated-inline-view-helper
Mar 13, 2026
Merged

Extract rule: template-deprecated-inline-view-helper#2450
NullVoxPopuli merged 1 commit intoember-cli:masterfrom
NullVoxPopuli:nvp/template-lint-extract-rule-template-deprecated-inline-view-helper

Conversation

@NullVoxPopuli
Copy link
Copy Markdown
Contributor

Split from #2371.

@NullVoxPopuli NullVoxPopuli force-pushed the nvp/template-lint-extract-rule-template-deprecated-inline-view-helper branch 3 times, most recently from 4991498 to 836c344 Compare March 6, 2026 21:05
@@ -0,0 +1,35 @@
# ember/template-deprecated-inline-view-helper

> **HBS Only**: This rule applies to classic `.hbs` template files only (loose mode). It is not relevant for `gjs`/`gts` files (strict mode), where these patterns cannot occur.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was deprecated in 1.x. I don't believe any 1.x project is going to be managing to use this plugin (it pre-dated npm), so should we bother porting this rule?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's easier for me to justify a migration path from ember-template-lint if an initial recommended-hbs is the same as in ember-template-lint

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess? I doubt any 1.x project was using that either tbh

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i agree it can be skipped.

if not:

  1. No isLocal check: {{#each items as |view|}}{{view.name}}{{/each}} would be falsely flagged.
  2. Missing dedup guard: No _processedByInlineViewHelper equivalent — same node could be reported twice (once from element visitor, once from mustache visitor).
  3. No autofix: Original provides fix suggestions (removing view. prefix).

@NullVoxPopuli NullVoxPopuli force-pushed the nvp/template-lint-extract-rule-template-deprecated-inline-view-helper branch from 836c344 to 09ba09c Compare March 10, 2026 22:23
@NullVoxPopuli NullVoxPopuli force-pushed the nvp/template-lint-extract-rule-template-deprecated-inline-view-helper branch from 09ba09c to 438984d Compare March 13, 2026 14:27
@NullVoxPopuli NullVoxPopuli merged commit 9df0806 into ember-cli:master Mar 13, 2026
11 checks passed
@NullVoxPopuli NullVoxPopuli deleted the nvp/template-lint-extract-rule-template-deprecated-inline-view-helper branch March 13, 2026 14:38
NullVoxPopuli added a commit that referenced this pull request Mar 14, 2026
Post-merge review of #2450 (`template-deprecated-inline-view-helper`)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants