Просмотр исходного кода

fix: return 1 instead of 0

no need for `(startingLineNumber || 1)`, when we can return 1 directly from `calculateStartingLine`
Dhanraj Padmashali 4 лет назад
Родитель
Сommit
49c19140e6
1 измененных файлов с 3 добавлено и 3 удалено
  1. 3 3
      index.js

+ 3 - 3
index.js

@@ -58,7 +58,7 @@ const calculateLinesToHighlight = (meta) => {
 }
 
 /**
- * Check if we want to start the line numbering from a given number or 0
+ * Check if we want to start the line numbering from a given number or 1
  * showLineNumbers=5, will start the numbering from 5
  * @param {string} meta
  * @returns {number}
@@ -72,7 +72,7 @@ const calculateStartingLine = (meta) => {
     } = RE.exec(meta)
     return Number(lines)
   }
-  return 0
+  return 1
 }
 
 /**
@@ -218,7 +218,7 @@ const rehypePrism = (options = {}) => {
     for (const [i, line] of codeLineArray.entries()) {
       // Code lines
       if (meta.toLowerCase().includes('showLineNumbers'.toLowerCase()) || options.showLineNumbers) {
-        line.properties.line = [(i + (startingLineNumber || 1)).toString()]
+        line.properties.line = [(i + startingLineNumber).toString()]
         line.properties.className.push('line-number')
       }