chore: remove unused eslint comments (#1809)

e2e/cases/bundler-chain/src/pre.js

@@ -1,4 +1,3 @@
-/* eslint-disable no-undef */
 const testEl = document.createElement('div');
 testEl.id = 'test-el';
 testEl.innerHTML = 'aaaaa';

e2e/cases/inline-chunk/src/another.js

@@ -4,6 +4,5 @@ import(
   /* webpackChunkName: "foo" */
   './foo'
 ).then(({ foo }) => {
-  // eslint-disable-next-line no-undef
   window.answer = `another ${foo()}`;
 });

e2e/cases/inline-chunk/src/index.js

@@ -1,12 +1,10 @@
 import './style.css';
 
-// eslint-disable-next-line no-undef
 window.test = 'aaaa';
 
 import(
   /* webpackChunkName: "foo" */
   './foo'
 ).then(({ foo }) => {
-  // eslint-disable-next-line no-undef
   window.answer = foo();
 });

e2e/cases/output/ascii/src/index.js

@@ -1,7 +1,5 @@
-// eslint-disable-next-line no-undef
 window.a = '你好 world!';
 
-// eslint-disable-next-line no-undef
 window.b = {
   Д: 'A',
   Å: 'A',

e2e/cases/output/externals/src/ex.js

@@ -1,2 +1 @@
-// eslint-disable-next-line no-undef
 window.aa = { a: 1 };

e2e/cases/preview/basic/src/index.js

@@ -1,2 +1 @@
-// eslint-disable-next-line no-undef
 document.getElementById('root').innerHTML = 'Hello Rsbuild!';

e2e/cases/react/remove-prop-types/src/App.jsx

@@ -12,7 +12,6 @@ App.propTypes = {
   testPropTypes: PropTypes.string,
 };
 
-// eslint-disable-next-line no-undef
 window.testAppPropTypes = App.propTypes;
 
 export default App;

e2e/cases/source/basic/src/pre.js

@@ -1,4 +1,3 @@
-/* eslint-disable no-undef */
 const testEl = document.createElement('div');
 testEl.id = 'test-el';
 testEl.innerHTML = 'aaaaa';

e2e/cases/svg/svg-default-export-component/src/App.jsx

@@ -1,4 +1,3 @@
-// eslint-disable-next-line no-unused-vars
 import Logo from './app.svg';
 
 function App() {

e2e/cases/svg/svg-external-react/src/App.jsx

@@ -1,4 +1,3 @@
-// eslint-disable-next-line no-unused-vars
 import svgImg, { ReactComponent as Logo } from './app.svg';
 import './App.css';
 

e2e/cases/svg/svg/src/App.jsx

@@ -1,4 +1,3 @@
-// eslint-disable-next-line no-unused-vars
 import svgImg, { ReactComponent as Logo } from './app.svg';
 import './App.css';
 

e2e/cases/svg/svgr-query-url/src/App.jsx

@@ -1,4 +1,3 @@
-// eslint-disable-next-line no-unused-vars
 import Logo from './app.svg';
 import svgImg from './app.svg?url';
 import './App.css';

e2e/cases/vue2/jsx-basic/src/index.js

@@ -1,7 +1,6 @@
 import Vue from 'vue';
 import A from './A';
 
-// eslint-disable-next-line no-new
 new Vue({
   el: '#root',
   render: (h) => h(A),

e2e/cases/vue2/sfc-basic/src/index.js

@@ -1,7 +1,6 @@
 import Vue from 'vue';
 import A from './A.vue';
 
-// eslint-disable-next-line no-new
 new Vue({
   el: '#root',
   render: (h) => h(A),

e2e/cases/vue2/sfc-lang-jsx/src/index.js

@@ -1,7 +1,6 @@
 import Vue from 'vue';
 import App from './App.vue';
 
-// eslint-disable-next-line no-new
 new Vue({
   el: '#root',
   render: (h) => h(App),

e2e/cases/vue2/sfc-lang-ts/src/index.js

@@ -1,7 +1,6 @@
 import Vue from 'vue';
 import App from './App.vue';
 
-// eslint-disable-next-line no-new
 new Vue({
   el: '#root',
   render: (h) => h(App, { props: { foo: 'bar' } }),

e2e/cases/vue2/sfc-lang-tsx/src/index.js

@@ -1,7 +1,6 @@
 import Vue from 'vue';
 import App from './App.vue';
 
-// eslint-disable-next-line no-new
 new Vue({
   el: '#root',
   render: (h) => h(App),

e2e/cases/vue2/sfc-style/src/index.js

@@ -2,7 +2,6 @@ import Vue from 'vue';
 import App from './App.vue';
 import './a.less';
 
-// eslint-disable-next-line no-new
 new Vue({
   el: '#root',
   render: (h) => h(App),

e2e/cases/wasm/wasm-basic/src/index.js

@@ -2,5 +2,4 @@ import { _Z4facti } from './factorial.wasm';
 
 const factorial = _Z4facti;
 
-// eslint-disable-next-line no-undef
 document.querySelector('#root').innerHTML = factorial(3);

e2e/cases/wasm/wasm-url/src/index.js

@@ -1,5 +1,3 @@
-// eslint-disable-next-line node/no-unsupported-features/node-builtins, node/prefer-global/url
 const wasmPath = new URL('./factorial.wasm', import.meta.url).pathname;
 
-// eslint-disable-next-line no-undef
 document.querySelector('#root').innerHTML = wasmPath;

e2e/scripts/shared.ts

@@ -29,7 +29,6 @@ export const gotoPage = async (
   return page.goto(url);
 };
 
-// eslint-disable-next-line @typescript-eslint/no-empty-function
 const noop = async () => {};
 
 export const createRsbuild = async (

packages/core/src/client/hmr/createSocketUrl.ts

@@ -37,7 +37,6 @@ export function formatURL({
   pathname: string;
 }) {
   if (typeof URL !== 'undefined') {
-    // eslint-disable-next-line node/prefer-global/url, node/no-unsupported-features/node-builtins
     const url = new URL('http://localhost');
     url.port = port;
     url.hostname = hostname;

packages/core/src/client/hmr/index.ts

@@ -18,10 +18,8 @@ let hasCompileErrors = false;
 
 function clearOutdatedErrors() {
   // Clean up outdated compile errors, if any.
-  // eslint-disable-next-line node/no-unsupported-features/node-builtins
   if (typeof console !== 'undefined' && typeof console.clear === 'function') {
     if (hasCompileErrors) {
-      // eslint-disable-next-line node/no-unsupported-features/node-builtins
       console.clear();
     }
   }

packages/plugin-babel/src/helper.ts

@@ -128,7 +128,6 @@ const modifyPresetOptions = <T>(
 export const getBabelUtils = (
   config: BabelTransformOptions,
 ): BabelConfigUtils => {
-  // eslint-disable-next-line @typescript-eslint/no-empty-function
   const noop = () => {};
 
   return {

packages/shared/src/utils.ts

@@ -78,7 +78,6 @@ export const awaitableGetter = <T>(
 ): AwaitableGetter<T> => {
   const then: PromiseLike<T[]>['then'] = (...args) =>
     Promise.all(promises).then(...args);
-  // eslint-disable-next-line no-thenable
   return { then, promises };
 };
 

scripts/check-changeset/src/index.ts

@@ -64,6 +64,5 @@ async function run() {
 
 run().catch((e) => {
   console.error(e);
-  // eslint-disable-next-line no-process-exit
   process.exit(1);
 });

scripts/update-packages/src/modern.ts

@@ -66,6 +66,5 @@ const updateModernVersion = async (
 
 run().catch((e) => {
   console.error(e);
-  // eslint-disable-next-line no-process-exit
   process.exit(1);
 });

scripts/update-packages/src/rspack.ts

@@ -67,6 +67,5 @@ const updateRspackVersion = async (
 
 run().catch((e) => {
   console.error(e);
-  // eslint-disable-next-line no-process-exit
   process.exit(1);
 });

scripts/update-packages/src/rspress.ts

@@ -38,6 +38,5 @@ async function run() {
 
 run().catch((e) => {
   console.error(e);
-  // eslint-disable-next-line no-process-exit
   process.exit(1);
 });