With Sorbet, can't make an activity or workflow class final
まだ誰も着手していません。
評価
調査の方向性
まず、issue にある strict Sorbet activity example で登録を再現し、次に #128 によって導入された重複と、それが影響する登録パスを調査します。完了条件は、final activities と workflows が method-not-final エラーなしで正常に登録されることです。
索引モデルが issue の本文から書いたものです。
説明
Activities marked final using Sorbet's final! fail on registration, with something like:
RuntimeError:
#<Class:0x00005558e0b25190> was declared as final but its method `execute` was not declared as final
(We/I like final classes, to prevent inadvertent implementation inheritance.)
This was broken by #128, which added the dup.
Minimal repro looks something like (off the top of my head):
# typed: strict
class MyActivity < Temporal::Activity
extend T::Sig
extend T::Helpers
final!
sig(:final) { void }
def execute; end
end
(and then trying to register the activity).
- 主要言語
- Ruby
- スター
- 288
- フォーク
- 113
- 平均マージ
- 10日 15時間
- マージ済み PR（30日）
- 2
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
coinbase/temporal-ruby のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
coinbase/temporal-ruby#361 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
coinbase/temporal-ruby#341 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
coinbase/temporal-ruby#328 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
coinbase/temporal-ruby#326 · コメント 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
coinbase/temporal-ruby#324 · コメント 3 件 · リアクション 2 件 ·
coinbase/temporal-ruby の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
googleapis/google-api-ruby-client#28001 · リアクション 5 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100