Skip to content

Linter: Autofix for actionview-prefer-collection-render #1964

Description

@marcoroth

I think it might need to be unsafe, but we already show the recommended way to rewrite in the offense message:

Example 1

<% @speakers.each do |speaker| %>
  <%= render speaker %>
<% end %>
Prefer `<%= render @speakers %>` over rendering a partial once per iteration. Collection rendering builds the partial once instead of for every item.Herb Linter [actionview-prefer-collection-render](https://herb-tools.dev/linter/rules/actionview-prefer-collection-render)

Example 2

<% @talks.each do |talk| %>
  <%= render "talk", talk: talk %>
<% end %>
Prefer `<%= render partial: "talk", collection: @talks %>` over rendering a partial once per iteration. Collection rendering builds the partial once instead of for every item.Herb Linter [actionview-prefer-collection-render](https://herb-tools.dev/linter/rules/actionview-prefer-collection-render)

Metadata

Metadata

Assignees

No one assigned

    Labels

    linter@herb-tools/linter for HTML+ERB templateslinter-autofixLinter autofix behaviorlinter-ruleIndividual linter rules and their documentation

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions