positive req/seq is not a warning

vercel · Jun 27, 2020 · 42d3443 · 42d3443
1 parent 479c38d
commit 42d3443
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/.github/actions/next-stats-action/src/add-comment.js b/.github/actions/next-stats-action/src/add-comment.js
@@ -101,7 +101,15 @@ module.exports = async function addComment(
             // check if there is still a change after rounding
             if (change !== 0) {
               const absChange = Math.abs(change)
-              change = `${change < 0 ? '-' : '⚠️ +'}${
+              const warnIfNegative = isBenchmark && itemKey.match(/req\/sec/)
+              const warn = warnIfNegative
+                ? change < 0
+                  ? ''
+                  : '⚠️ '
+                : change > 0
+                ? '⚠️ '
+                : ''
+              change = `${warn}${change < 0 ? '-' : '+'}${
                 useRawValue ? absChange : prettify(absChange, prettyType)
               }`
             }