Skip to content
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

require-jsdoc on module export arrow function #551

Closed
ellioseven opened this issue May 26, 2020 · 1 comment
Closed

require-jsdoc on module export arrow function #551

ellioseven opened this issue May 26, 2020 · 1 comment

Comments

@ellioseven
Copy link

ellioseven commented May 26, 2020

Expected behavior

Rules:

{
  "rules": {
    "jsdoc/require-jsdoc": [
      "error",
      {
        "publicOnly": false,
        "require": {
          "FunctionExpression": true,
          "ArrowFunctionExpression": true,
          "FunctionDeclaration": true,
          "MethodDefinition": true
        }
      }
    ]
  }
}

Code:

module.exports.foo = (bar) => {
  return bar + "biz"
}

Error:

// error   Missing JSDoc comment    jsdoc/require-jsdoc

Perhaps I'm missing something here? But I can't seem to get an error to trigger. I've looked at the examples and tried the issue queue but I can't seem to get this to work. Is this a bug or am I doing something wrong?

Actual behavior

No error is formed.

Environment

  • Node version: 10.15.3
  • ESLint version 7.1.0
  • eslint-plugin-jsdoc version: 25.4.2
@ellioseven ellioseven added the bug label May 26, 2020
@ellioseven ellioseven changed the title require-jsdoc on module export function require-jsdoc on module export arrow function May 26, 2020
@gajus
Copy link
Owner

gajus commented May 26, 2020

🎉 This issue has been resolved in version 25.4.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@gajus gajus added the released label May 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants