fix(require-jsdoc): have reporting cover entire first line of function so more visible; (causes endLine to be removed); #386

brettz9 · brettz9 · commit 35583774fb14 · 2019-09-27T22:42:18.000+08:00
diff --git a/src/rules/requireJsdoc.js b/src/rules/requireJsdoc.js
@@ -174,7 +174,7 @@ export default {
 
       const report = () => {
         const loc = {
-          end: node.loc.start,
+          end: node.loc.start + 1,
           start: node.loc.start,
         };
         context.report({
diff --git a/test/rules/assertions/requireJsdoc.js b/test/rules/assertions/requireJsdoc.js
@@ -263,7 +263,7 @@ export default {
           }`,
       errors: [
         {
-          endLine: 2,
+          endLine: undefined,
           line: 2,
           message: 'Missing JSDoc comment.',
         },

Original file line number	Diff line number	Diff line change
`@@ -263,7 +263,7 @@ export default {`
`263`	`263`	}`,
`264`	`264`	`errors: [`
`265`	`265`	`{`
`266`		`- endLine: 2,`
	`266`	`+ endLine: undefined,`
`267`	`267`	`line: 2,`
`268`	`268`	`message: 'Missing JSDoc comment.',`
`269`	`269`	`},`