New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

`valid-jsdoc` false positive: an object pattern in an assignment pattern. #6097

Closed
mysticatea opened this Issue May 6, 2016 · 1 comment

Comments

Projects
None yet
1 participant
@mysticatea
Member

mysticatea commented May 6, 2016

What version of ESLint are you using?

master branch.

What parser (default, Babel-ESLint, etc.) are you using?

default.

Please show your full configuration:

nothing.

What did you do? Please include the actual source code causing the issue.

/**
 * @param {string} a - a.
 * @param {object} [obj] - obj.
 * @param {string} obj.b - b.
 * @param {string} obj.c - c.
 * @returns {void}
 */
function foo(a, {b, c} = {}) {
    // empty
}
> eslint test.js --env es6 --rule valid-jsdoc:2 --no-ignore --no-eslintrc

C:\Users\starc\Documents\GitHub\eslint\test.js
  1:1  error  Expected JSDoc for 'undefined' but found 'obj'  valid-jsdoc

✖ 1 problem (1 error, 0 warnings)

What did you expect to happen?

No error.

What actually happened? Please include the actual, raw output from ESLint.

  1:1  error  Expected JSDoc for 'undefined' but found 'obj'  valid-jsdoc
@mysticatea

This comment has been minimized.

Show comment
Hide comment
@mysticatea

mysticatea May 14, 2016

Member

I'm working on this.

Member

mysticatea commented May 14, 2016

I'm working on this.

@mysticatea mysticatea self-assigned this May 14, 2016

mysticatea added a commit that referenced this issue May 14, 2016

chriswong added a commit to ecomfe/fecs that referenced this issue Jun 7, 2016

@eslint eslint bot locked and limited conversation to collaborators Feb 6, 2018

@eslint eslint bot added the archived due to age label Feb 6, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.