capistrano / capistrano/sshkit

Nested "on" blocks should raise or warn

Offen
#352 4 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
discuss! enhancement help wanted
Vorherrschende Sprache
Ruby
Sterne
1.2k
Forks
257
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

From https://github.com/capistrano/sshkit/issues/348#issuecomment-220160838

> AFAIK nested on blocks were never considered in the design. The fact that it works at all is really just coincidence, due to how the SSHKit DSL is mixed into all objects. IMO it should raise an exception. You can open an SSHKit issue.

My personal opinion is that we should actually support it (take intersection of all role sets in the call stack), but until then, we should raise or warn when we find a nested `on` blocks.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne damit, das im Issue beschriebene Verhalten des verschachtelten `on`-Blocks zu reproduzieren, und untersuche, wie die SSHKit DSL verschachtelte Aufrufe behandelt. Erledigt ist dies, wenn verschachtelte `on`-Blöcke nicht mehr stillschweigend fortfahren, sondern stattdessen konsistent eine Exception auslösen oder eine Warnung ausgeben, wobei das gewählte Verhalten in Tests abgebildet ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
ruby
Bereich
devops
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

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