Reconsider capIsNew: false for new-cap
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 35/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- eslint, javascript
- 領域
- tooling
調査の方向性
まず、#1090 の new-cap 設定変更と、リンクされている Airbnb issue の根拠を確認し、その後 [email protected] で示されているクラスと関数呼び出しの例を再現します。capIsNew を元に戻す場合と capIsNewExceptions または capIsNewExceptionPattern を使用する場合を比較し、#1106 の関連する提案も検討します。意図した動作と設定について合意され、プロジェクトの関連するチェックでカバーされれば完了です。
索引モデルが issue の本文から書いたものです。
説明
With [email protected] the code
class Example {}
module.exports = Example();
would result in the error:
example.js:2:18: A function with a name starting with an uppercase letter should only be used as a constructor. [Error/new-cap]
As a result of #1090, which set capIsNew: false for new-cap, this is no longer the case.
https://github.com/airbnb/javascript/issues/1089#issuecomment-249409624 which motivated the change only mentions adding "capIsNewExceptions": ["Immutable.Map", "Immutable.Set", "Immutable.List"]. Would it make sense to revert capIsNew: false (and perhaps adopt "capIsNewExceptionPattern": "^Immutable.\\w" as suggested in #1106) or was there another rationale for setting capIsNew: false?
Thanks,
Kevin
- 主要言語
- JavaScript
- スター
- 148k
- フォーク
- 26.6k
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
airbnb/javascript のほかの issue
-
Severity: Unhandled promise rejection in `whitespace-async.js` when ESLint async path is used対応中かも @bodapatisaikrishna が 24 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
airbnb/javascript#3237 · コメント 6 件 ·
-
Inconsistent semicolon usage in examples (Arrays vs Functions)対応中かも @Developer-shivamMishra が 15 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
airbnb/javascript#3152 · コメント 4 件 ·
-
No error handling around execSync + JSON.parse in whitespace.js (ESLint 9 path)対応中かも @dataCenter430 が 219 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 48/100
airbnb/javascript#3238 · コメント 8 件 ·
-
Upgrading eslint-plugin-react-hooks対応中かも @weihongyu12 が 365 日前に担当しました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 38/100
airbnb/javascript#3186 · コメント 3 件 · リアクション 2 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
airbnb/javascript#3173 · コメント 9 件 · リアクション 1 件 ·
airbnb/javascript の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
XRPLF/xrpl-dev-portal#4000 ·
メンテナーはふだん 1 日以内に返信
-
accessibility good first issue
難易度 1/5 1〜3時間 初心者へのやさしさ 92/100
メンテナーはふだん 1 日以内に返信
-
[BUG] Bubble chart series name is not XML-escaped in the embedded workbook (xl/tables/table1.xml)オープン
難易度 1/5 1時間未満 初心者へのやさしさ 94/100
-
Content:Learn needs triage
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
enhancement good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
anoopcodehack/DevBoard#609 ·
メンテナーはふだん 1 日以内に返信