Rollup merge of rust-lang#142618 - GuillaumeGomez:eslint-no-console, r=lolbinarycat

jhpratt · web-flow · commit a47b364dc62b · 2025-06-17T23:19:38.000+02:00
Lint about `console` calls in rustdoc JS As discussed [here](rust-lang#142100 (comment)), this PR enforces that `console` is not used in rustdoc JS by default. cc `@lolbinarycat`
diff --git a/src/librustdoc/html/static/.eslintrc.js b/src/librustdoc/html/static/.eslintrc.js
@@ -91,5 +91,6 @@ module.exports = {
         "no-script-url": "error",
         "no-sequences": "error",
         "no-div-regex": "error",
+        "no-console": "error",
     }
 };
diff --git a/src/librustdoc/html/static/js/search.js b/src/librustdoc/html/static/js/search.js
@@ -1133,6 +1133,7 @@ class RoaringBitmap {
         }
         for (let j = 0; j < size; ++j) {
             if (offsets && offsets[j] !== i) {
+                // eslint-disable-next-line no-console
                 console.log(this.containers);
                 throw new Error(`corrupt bitmap ${j}: ${i} / ${offsets[j]}`);
             }

Original file line number	Diff line number	Diff line change
`@@ -91,5 +91,6 @@ module.exports = {`
`91`	`91`	`"no-script-url": "error",`
`92`	`92`	`"no-sequences": "error",`
`93`	`93`	`"no-div-regex": "error",`
	`94`	`+ "no-console": "error",`
`94`	`95`	`}`
`95`	`96`	`};`
Original file line number	Diff line number	Diff line change
`@@ -1133,6 +1133,7 @@ class RoaringBitmap {`
`1133`	`1133`	`}`
`1134`	`1134`	`for (let j = 0; j < size; ++j) {`
`1135`	`1135`	`if (offsets && offsets[j] !== i) {`
	`1136`	`+ // eslint-disable-next-line no-console`
`1136`	`1137`	`console.log(this.containers);`
`1137`	`1138`	throw new Error(`corrupt bitmap ${j}: ${i} / ${offsets[j]}`);
`1138`	`1139`	`}`