hyperstack-org/hyperstack

Hyperspec to_js method broken

Open

#393 建立於 2021年3月31日

在 GitHub 查看
 (0 留言) (0 反應) (0 負責人)JavaScript (531 star) (43 fork)batch import
buggood first issue

描述

    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

貢獻者指南

技術棧
無
領域
testing
議題類型
bug
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
1
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
under 1 hour
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
stale
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
clear
前置要求
Basic RubyOpal basics
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
90
研究方向
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.