ember-cli / ember-cli/loader.js

Support For define.amd

オープン
#60 コメント 11 件 リアクション 3 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
79
フォーク
69
PR マージ指標
30日以内にマージされた PR はありません

説明

Trying to use rollup for a refactor of Backburner with @krisselden and others. I noticed that rollup has AMD check looks for `define && define.amd`. Loader.js explicitly opts out of this with `define.petals`, wondering if people would be opposed to just setting `define.amd` to `true`. It's not spec compliant, however loader.js is already not spec compliant in this regard.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

Loader.js から始め、現在 loader がどのように define.petals を公開し、define.amd のチェックを回避しているかを確認してください。互換性の挙動を変更する前に issue の議論を確認し、その後、既存の AMD の挙動を壊さずに Rollup が loader を認識することを検証してください。意図した define.amd サポートが実装され、その互換性上のトレードオフがプロジェクトで利用可能なチェックによってカバーされれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript
領域
tooling
issue の種類
機能追加
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
38/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。