Raise error/warning when using `:if` on a slotable component
まだ誰も着手していません。
評価
調査の方向性
Start in lib/surface/compiler.ex around the referenced directive handling and read the discussion in issue 182. Determine how slotable or renderless components are compiled and compare their accepted directives with , which currently accepts :let. Done means incompatible directives such as :for produce the requested compile-time error or warning while supported directives remain valid.
索引モデルが issue の本文から書いたものです。
説明
We should raise a compile-time error if the user tries to use :for or any incompatible directive on a renderless component.
It should accept the same directives as <template>, which currently is just :let:https://github.com/msaraiva/surface/blob/b745e1322ce7b20fcbcf8503f507349da0772435/lib/surface/compiler.ex#L40.
Initial discussion started at https://github.com/msaraiva/surface/issues/182.
- 主要言語
- Elixir
- スター
- 2.1k
- フォーク
- 146
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
surface-ui/surface のほかの issue
-
難易度 1/5 1〜3時間 初心者へのやさしさ 78/100
surface-ui/surface#634 · リアクション 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
surface-ui/surface#770 ·
-
Website is downオープン
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
surface-ui/surface#768 · コメント 4 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
surface-ui/surface#762 · コメント 2 件 · リアクション 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
surface-ui/surface#759 · リアクション 1 件 ·
surface-ui/surface の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
electric-sql/electric#4830 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信
-
L: java:gradle L: java:maven L: ruby:bundler T: bug 🐞
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
dependabot/dependabot-core#16547 ·
メンテナーはふだん 1 日以内に返信
-
github_actions
難易度 1/5 1時間未満 初心者へのやさしさ 94/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
team-alembic/clarity#146 ·