Skip to content

Commit cdbd468

Browse files
l0lawrenceCopilot
andcommitted
build(http-client-python): migrate lint to oxlint
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: d9d0e1a1-ea80-4398-87be-49658388a53d
1 parent 3f20bff commit cdbd468

4 files changed

Lines changed: 74 additions & 1243 deletions

File tree

packages/http-client-python/eng/scripts/ci/config/eslint-ci.config.mjs

Lines changed: 0 additions & 64 deletions
This file was deleted.

packages/http-client-python/eng/scripts/ci/lint.ts

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ ${pc.bold("Description:")}
4141
4242
${pc.bold("Options:")}
4343
${pc.cyan("-e, --emitter")}
44-
Run ESLint on TypeScript emitter code.
44+
Run Oxlint on TypeScript emitter code.
4545
4646
${pc.cyan("-g, --generator")}
4747
Run pylint on Python generator source code (pygen).
@@ -85,7 +85,7 @@ function runCommand(
8585
const workingDir = cwd || root;
8686
const name = displayName || command;
8787

88-
// Add node_modules/.bin directories to PATH so commands like eslint can be found
88+
// Add node_modules/.bin directories to PATH so commands like oxlint can be found
8989
// Also set NODE_PATH so that config files can resolve packages from monorepo's node_modules
9090
const pathSep = process.platform === "win32" ? ";" : ":";
9191
const binPaths = [
@@ -134,20 +134,17 @@ function runCommand(
134134

135135
async function lintEmitter(): Promise<boolean> {
136136
console.log(`\n${pc.bold("=== Linting TypeScript ===")}\n`);
137-
// Run eslint with local config to avoid dependency on monorepo's eslint.config.js
138-
// This ensures the package can be linted in CI without full monorepo dependencies
139-
// Lint both emitter/ and eng/scripts/ directories
140137
return runCommand(
141-
"eslint",
138+
"oxlint",
142139
[
143140
"emitter/",
144141
"eng/scripts/",
145142
"--config",
146-
"eng/scripts/ci/config/eslint-ci.config.mjs",
147-
"--max-warnings=0",
143+
"../../oxlint.config.ts",
144+
"--deny-warnings",
148145
],
149146
root,
150-
"eslint emitter/ eng/scripts/ --max-warnings=0",
147+
"oxlint emitter/ eng/scripts/ --deny-warnings",
151148
);
152149
}
153150

0 commit comments

Comments
 (0)