hyperstack-org/hyperstack

Hyperspec to_js method broken

Open

#393 geöffnet am 31. März 2021

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (531 Stars) (43 Forks)batch import
buggood first issue

Beschreibung

    def to_js(*args, &block)
      opal_compile(*process_params(*args, &block))
    end

needs to be

    def to_js(*args, &block)
      opal_compile(process_params(*args, &block).first)
    end

(second value from process_params is options for mounting, which opal_compile doesn't need)

Should have a simple spec anyway that says yes I returned a string with length > 0

Contributor Guide

Tech Stack
Keine
Domain
testing
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Basic RubyOpal basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
90
Research-Richtung
Apply the one line fix in the `to js` method: wrap `process params(*args, &block)` with `.first` to pass only the first argument to `opal compile`. Then add a simple spec that verifies `to js` returns a non empty string. Locate the file containing the method (likely in the Hyperspec source) and make the change, then write a test to confirm the behavior.