Reconsider capIsNew: false for new-cap
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- eslint, javascript
- Domaine
- tooling
Piste de recherche
Commencez par examiner la modification de configuration de new-cap de #1090 et la justification dans l’issue Airbnb liée, puis reproduisez l’exemple montré de classe et d’appel de fonction avec [email protected]. Comparez le fait de rétablir capIsNew avec l’utilisation de capIsNewExceptions ou de capIsNewExceptionPattern, et prenez en compte la proposition associée dans #1106. C’est terminé lorsque le comportement et la configuration attendus sont définis d’un commun accord et couverts par les vérifications pertinentes du projet.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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
- Langage dominant
- JavaScript
- Étoiles
- 148k
- Forks
- 26.6k
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de airbnb/javascript
-
Severity: Unhandled promise rejection in `whitespace-async.js` when ESLint async path is usedPeut-être pris @bodapatisaikrishna l’a pris il y a 26 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
airbnb/javascript#3237 · 6 commentaires ·
-
Inconsistent semicolon usage in examples (Arrays vs Functions)Peut-être pris @Developer-shivamMishra l’a pris il y a 16 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
airbnb/javascript#3152 · 4 commentaires ·
-
No error handling around execSync + JSON.parse in whitespace.js (ESLint 9 path)Peut-être pris @dataCenter430 l’a pris il y a 221 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 48/100
airbnb/javascript#3238 · 8 commentaires ·
-
Upgrading eslint-plugin-react-hooksPeut-être pris @weihongyu12 l’a pris il y a 367 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 38/100
airbnb/javascript#3186 · 3 commentaires · 2 réactions ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
airbnb/javascript#3173 · 9 commentaires · 1 réaction ·
Toutes les issues de airbnb/javascript
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 60/100
quarto-dev/quarto-cli#15042 ·
Les mainteneurs répondent en général sous 1 jour
-
OpenCode install copies the Pi harness directory (`.pi/`) into the OpenCode config rootPeut-être pris @gaurav0107 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Les mainteneurs répondent en général sous 3 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
NaturalIntelligence/fast-xml-parser#890 · 1 commentaire ·
Les mainteneurs répondent en général sous 3 jours
-
Docs viewer: inverted grey colormap, Phantom CT window -20 to 35 HU, smoothing up to 40 mmPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouvertearea: docs enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
PyTomography/PyTomography#331 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
pnpm/get.pnpm.io#68 ·