Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

Exception in no-redundant-jsdoc #3424

Closed
manu-st opened this issue Oct 31, 2017 · 1 comment
Closed

Exception in no-redundant-jsdoc #3424

manu-st opened this issue Oct 31, 2017 · 1 comment

Comments

@manu-st
Copy link

manu-st commented Oct 31, 2017

Bug Report

  • TSLint version: 5.8
  • TypeScript version: 2.5.3
  • Running TSLint via: CLI

TypeScript code being linted

/**
 * Desc.
 *
 * @class Simple
 */
export class Simple
{
   /**
    * Name.
    *
    * @memberof Simple
    */
   name: string;
}

with tslint.json configuration:

{
   "rules":
   {
       "no-redundant-jsdoc": true
   }
}

Actual behavior

Error: Unexpected tag kind: JSDocClassTag
    at checkTag (C:\Dev\node_modules\tslint\lib\rules\noRedundantJsdocRule.js:99:23)
    at cb (C:\Dev\node_modules\tslint\lib\rules\noRedundantJsdocRule.js:62:25)
    at Array.forEach (<anonymous>)
    at walk (C:\Dev\node_modules\tslint\lib\rules\noRedundantJsdocRule.js:55:34)
    at Rule.AbstractRule.applyWithFunction (C:\Dev\node_modules\tslint\lib\language\rule\abstractRule.js:39:9)
    at Rule.apply (C:\Dev\node_modules\tslint\lib\rules\noRedundantJsdocRule.js:35:21)
    at Linter.applyRule (C:\Dev\node_modules\tslint\lib\linter.js:177:29)
    at C:\Dev\node_modules\tslint\lib\linter.js:119:85
    at Object.flatMap (C:\Dev\node_modules\tslint\lib\utils.js:151:29)
    at Linter.getAllFailures (C:\Dev\node_modules\tslint\lib\linter.js:119:32)

Expected behavior

ERROR: C:/Dev/Simple.ts[4, 4]: JSDoc tag '@class' is redundant in TypeScript code.
ERROR: C:/Dev/Simple.ts[11, 8]: JSDoc tag '@memberof' is redundant in TypeScript code.
@manu-st manu-st changed the title Crash in no-redundant-jsdoc Exception in no-redundant-jsdoc Oct 31, 2017
@ajafff
Copy link
Contributor

ajafff commented Oct 31, 2017

Duplicate of #3413
I already submitted a PR to fix this

@ajafff ajafff closed this as completed Oct 31, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants