ember-cli / ember-cli/loader.js
Support For define.amd
- 主要言語
- 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