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

no-multi-asterisks should not trigger when there's whitespace between asterisks #803

Closed
edg2s opened this issue Nov 2, 2021 · 4 comments

Comments

@edg2s
Copy link
Contributor

edg2s commented Nov 2, 2021

Expected behavior

The sample code is allowed when using no-multi-asterisks.

Actual behavior

The code is not allowed.

ESLint Config

{ "rules": {
  "no-multi-asterisks": "error"
} }

ESLint sample

/**
 * The method does 2 things:
 * * Thing 1
 * * Thing 2
 **/
/**
 * This muti-line comment contains some
 * *non-standard bold* syntax
 **/

Environment

  • Node version:
  • ESLint version
  • eslint-plugin-jsdoc version: 37.0.3
@brettz9
Copy link
Collaborator

brettz9 commented Dec 18, 2021

I think we need some config for this because this could be working as intended (to catch stray asterisks at the beginning).

brettz9 added a commit to brettz9/eslint-plugin-jsdoc that referenced this issue Dec 29, 2021
brettz9 added a commit to brettz9/eslint-plugin-jsdoc that referenced this issue Dec 29, 2021
@brettz9
Copy link
Collaborator

brettz9 commented Dec 29, 2021

I would guess it may need some tweaking/refactoring, but take a look at #803 #828

@brettz9
Copy link
Collaborator

brettz9 commented Dec 30, 2021

Sorry had the wrong issue there... Meant #828

@gajus
Copy link
Owner

gajus commented Dec 31, 2021

🎉 This issue has been resolved in version 37.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

3 participants