Try eslint-plugin-require-jsdoc-except drop-in require-jsdoc ESLint rule

[ntwb]

Description

Have ESLint not throw require-jsdoc for methods.

Try eslint-plugin-require-jsdoc-except drop-in require-jsdoc ESLint rule

Reference: #4504 #4245

Bumped into this https://github.com/MaienM/eslint-plugin-require-jsdoc-except require-jsdoc drop-in rule that allows you to exclude certain methods from requiring a JSDoc.

How Has This Been Tested?

run npm run lint

I've also left in the "ignore": ["constructor", "render"] option for completeness though it's not required as MethodDefinition has been defined false

Screenshots (jpeg or gifs if applicable):

Types of changes

ESLint JSDoc linting

Checklist:

• My code is tested.
• My code follows the WordPress code style.
• My code follows has proper inline documentation.

[pento]

Nice. The warnings are still super spammy, though - can they be moved out of the npm run lint command for now? I don't mind there being a separate command for whoever wants to go through and fix them all, but they shouldn't be getting in the way of day-to-day work.

[ntwb]

Let's merge #4510 now and get rid of these warnings for now and follow up by way of discussion this PRs merits over the coming days and iterate as/if needed

[ntwb]

FYI: Don't merge this PR please, we should look to moving any JSDoc ESLint linting to a separate lint task that doesn't affect other contributors

[aduth]

@ntwb Will you be able to revisit this one?

[ntwb]

@ntwb Will you be able to revisit this one?

Apologies @aduth, I missed this, a new PR for this is available in #6341