github / github/rubocop-github

Unify controller render cops?

Offen
#34 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Ruby
Sterne
667
Forks
164
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

It looks like there are several cops which implement various approaches to recommending:

```ruby
render "path/to/template", { ... options ... }
```

Why are these cops separate? Could they be unified?

For example, one cop replaces `render action: :edit` => `render action: "edit"`, but another replaces `render action: "edit"` => `render "edit"`. So, could we jump right from `render action: :edit"` => `render "edit"`?

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

The issue does not name files or tests. Start by locating the controller render cops that handle symbol and string actions, then compare their recommendations and correction behavior. Done means determining whether the cops can be unified and defining a consistent direct correction from `render action: :edit` to `render "edit"`.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
ruby
Bereich
tooling
Issue-Typ
Refactoring
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.