Disable import/no-cycle rule to save linting time

Author: leeyi45

.github/actions/src/info/__tests__/index.test.ts

@@ -7,6 +7,15 @@ import { getAllPackages, getRawPackages } from '../index.js';
 
 const mockedCheckChanges = vi.spyOn(git, 'checkForChanges');
 
+vi.mock(import('path'), async importOriginal => {
+  const { posix } = await importOriginal();
+
+  return {
+    default: posix,
+    posix,
+  };
+});
+
 class NodeError extends Error {
   constructor(public readonly code: string) {
     super();

eslint.config.js

@@ -226,7 +226,8 @@ export default tseslint.config(
     rules: {
       'import/first': 'warn',
       'import/newline-after-import': 'warn',
-      'import/no-cycle': 'error',
+      // This rule is very time intensive.
+      // 'import/no-cycle': 'error',
       'import/no-duplicates': ['warn', { 'prefer-inline': false }],
       'import/no-useless-path-segments': 'error',
       'import/order': [

lib/vitest-reporter/src/__tests__/coverage.test.ts

@@ -1,10 +1,10 @@
 import fs from 'fs';
 import { describe, expect, it as baseIt, vi } from 'vitest';
-import reporter from '../coverage-reporter.cjs'
+import reporter from '../coverage-reporter.cjs';
 
 // I really don't know why importing from a CJS module doesn't work here
 // so that's why this line is here
-const CoverageReporter = reporter as any
+const CoverageReporter = reporter as any;
 
 vi.spyOn(fs, 'createWriteStream').mockReturnValue({
   write: () => true,
@@ -34,7 +34,7 @@ describe(CoverageReporter, () => {
   });
 
   it.skipIf(!!process.env.GITHUB_STEP_SUMMARY)('doesn\'t open the summary file if GITHUB_STEP_SUMMARY isn\'t defined', ({ reporter }) => {
-      reporter.onStart({} as any, { watermarks: {} } as any);
-      expect(fs.createWriteStream).not.toHaveBeenCalled();
+    reporter.onStart({} as any, { watermarks: {} } as any);
+    expect(fs.createWriteStream).not.toHaveBeenCalled();
   });
 });

lib/vitest-reporter/src/coverage-reporter.cts

@@ -7,9 +7,9 @@
 
 // This is some weird typescript only syntax that allows us to continue to have
 // the type information even though we're using CJS imports
-import fs = require('fs')
-import report = require('istanbul-lib-report');
+import fs = require('fs');
 import type { CoverageSummary } from 'istanbul-lib-coverage';
+import report = require('istanbul-lib-report');
 
 /**
  * Determines if the coverage summary has full coverage