Clobbering `this` from `options` might be murky
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 25/100
- issue の種類
- リファクタリング
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- javascript
調査の方向性
core-object.js の18–20行目にあるコンストラクターから始め、基底クラスが options hash からプロパティを割り当てる方法を確認します。そのコンストラクターを空にする場合の互換性と移行への影響を判断します。breaking-change の判断と想定される動作が文書化され、影響を受けるテストが特定されていれば完了です。
索引モデルが issue の本文から書いたものです。
説明
If I understand the constructor correctly, we're providing the ability to assign arbitrary properties on this through an options hash as a common base class constructor.
I used to do this in early versions of Broccoli plugin classes because it seemed convenient. I stopped doing it when I started running into cases where some library upgrade would cause property names to start colliding with option names. I simply do this.options = options in constructors ever since, and have been much happier with that.
My suggestion would be to make the base class constructor empty, and let people do their own option handling. I realize this might be a breaking change, but perhaps it's worth considering for the next major version?
- 主要言語
- JavaScript
- スター
- 16
- フォーク
- 16
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
ember-cli/core-object のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 25/100
ember-cli/core-object#35 · リアクション 1 件 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 15/100
ember-cli/core-object#29 ·
ember-cli/core-object の issue をすべて見る
似ている issue
-
Mend: dependency security vulnerability untriaged
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
opensearch-project/security-dashboards-plugin#2543 ·
メンテナーはふだん 1 日以内に返信
-
[quality] refresh-radar-reports.yml runs on ubuntu-latest while every other job pins ubuntu-24.04オープンagent/quality hive/hosted-available-lke648397-260827-5n31 quality testing
難易度 1/5 1〜3時間 初心者へのやさしさ 90/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
godotengine/godot-website#1432 ·
-
Add: Mooz Retroオープンchannels:add check:passed
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
メンテナーはふだん 2 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
メンテナーはふだん 1 日以内に返信