Reconsider capIsNew: false for new-cap
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- eslint, javascript
- Bereich
- tooling
Rechercherichtung
Beginne mit der Prüfung der Änderung an der new-cap-Konfiguration aus #1090 und der Begründung im verknüpften Airbnb-Issue. Reproduziere anschließend das gezeigte Beispiel für eine Klasse und einen Funktionsaufruf mit [email protected]. Vergleiche das Zurücksetzen von capIsNew mit der Verwendung von capIsNewExceptions oder capIsNewExceptionPattern und berücksichtige den verwandten Vorschlag in #1106. Die Aufgabe ist abgeschlossen, wenn das beabsichtigte Verhalten und die Konfiguration vereinbart und durch die relevanten Checks des Projekts abgedeckt sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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
- Vorherrschende Sprache
- JavaScript
- Sterne
- 148k
- Forks
- 26.5k
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus airbnb/javascript
-
Severity: Unhandled promise rejection in `whitespace-async.js` when ESLint async path is usedEvtl. vergeben @bodapatisaikrishna hat das vor 33 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
airbnb/javascript#3237 · 6 Kommentare ·
-
Inconsistent semicolon usage in examples (Arrays vs Functions)Evtl. vergeben @Developer-shivamMishra hat das vor 23 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
airbnb/javascript#3152 · 4 Kommentare ·
-
No error handling around execSync + JSON.parse in whitespace.js (ESLint 9 path)Evtl. vergeben @dataCenter430 hat das vor 228 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 48/100
airbnb/javascript#3238 · 8 Kommentare ·
-
Upgrading eslint-plugin-react-hooksEvtl. vergeben @weihongyu12 hat das vor 374 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 38/100
airbnb/javascript#3186 · 3 Kommentare · 2 Reaktionen ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
airbnb/javascript#3173 · 9 Kommentare · 1 Reaktion ·
Alle Issues in airbnb/javascript
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
Maintainer antworten meist innerhalb von 1 Tag
-
status: waiting for maintainer
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
mui/material-ui#49511 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
mpetazzoni/sse.js#122 ·
-
[eslint-plugin] valid-styles rejects <progress> pseudo-elements (::-webkit-progress-bar, ::-webkit-progress-value, ::-moz-progress-bar)Evtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
Maintainer antworten meist innerhalb von 2 Tagen
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
swc-project/swc-next#1022 ·
Maintainer antworten meist innerhalb von 1 Tag