microsoft / microsoft/TypeScript
Consecutive function declaration problems in JS files
Personne n'a encore pris cette issue.
- Langage dominant
- Go
- Étoiles
- 111k
- Forks
- 14.4k
- Merge moyen
- 1 j 19 h
- PR mergées (30 j)
- 117
Description
Bug Report
🔎 Search Terms
function declaration, method declaration, consecutive, comma
🕗 Version & Regression Information
- This is the behavior in every version I tried, and I reviewed the FAQ for entries about jsdoc problems
Tested with typescript@4.4.0-dev.20210602
⏯ Playground Link
💻 Code
/**
* @param {number} num
* @returns {number}
*/
const numb = (num) => num,
/**
* @param {string} str
* @returns {string}
*/
str = (str) => str
console.log(numb(2), str(4))
🙁 Actual behavior
Because of the consecutive declaration the real error is not catched. This only happens in js files
🙂 Expected behavior
Method declaration style shouldn't matter in js files either, so the error should be: ""Argument of type 'number' is not assignable to parameter of type 'string'."
Guide de contribution
Ouvrir le guide de contribution
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.
Piste de recherche
Exécutez la reproduction Playground liée avec le code JavaScript fourni et confirmez que les déclarations consécutives séparées par des virgules suppriment le diagnostic de type d’argument attendu. Trouvez le chemin de vérification des types JavaScript et ajoutez un test de régression pour l’exemple ; c’est terminé lorsqu’il signale que number n’est pas assignable à string, conformément au comportement attendu.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, typescript
- Domaine
- compilers
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100