Skip to content

Commit 1a54933

Browse files
authored
Revert "Use main eslint config for copilot extension too"
1 parent f9f1cdc commit 1a54933

45 files changed

Lines changed: 3396 additions & 2216 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.eslint-ignore

Lines changed: 2 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,25 +1,9 @@
11
**/build/*/**/*.js
2-
**/dist/**/*
2+
**/dist/**/*.js
33
**/extensions/**/*.d.ts
44
**/extensions/**/build/**
55
**/extensions/**/colorize-fixtures/**
6-
**/extensions/copilot/coverage/**
7-
**/extensions/copilot/.esbuild/**
8-
**/extensions/copilot/.simulation/**
9-
**/extensions/copilot/.eslintplugin/**
10-
**/extensions/copilot/chat-lib/**
11-
**/extensions/copilot/test/simulation/fixtures/**
12-
**/extensions/copilot/test/scenarios/**
13-
**/extensions/copilot/test/aml/out/**
14-
**/extensions/copilot/src/util/vs/**
15-
**/extensions/copilot/src/platform/parser/test/node/fixtures/**
16-
**/extensions/copilot/src/extension/test/node/fixtures/**
17-
**/extensions/copilot/src/extension/prompts/node/test/fixtures/**
18-
**/extensions/copilot/src/extension/typescriptContext/serverPlugin/fixtures/**
19-
**/extensions/copilot/src/extension/typescriptContext/serverPlugin/lib/**
20-
**/extensions/copilot/src/extension/typescriptContext/serverPlugin/dist/**
21-
**/extensions/copilot/src/extension/completions-core/**/testdata/*
22-
**/extensions/copilot/.vscode/extensions/test-extension/dist/**
6+
**/extensions/copilot/**
237
**/extensions/css-language-features/server/test/pathCompletionFixtures/**
248
**/extensions/html-language-features/server/lib/jquery.d.ts
259
**/extensions/html-language-features/server/src/test/pathCompletionFixtures/**
@@ -52,5 +36,4 @@
5236
**/test/automation/out/**
5337
**/typings/**
5438
**/.build/**
55-
**/.vscode-test/**
5639
!.vscode

.github/workflows/pr.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -288,6 +288,10 @@ jobs:
288288
working-directory: extensions/copilot
289289
run: npm run typecheck
290290

291+
- name: Lint
292+
working-directory: extensions/copilot
293+
run: npm run lint
294+
291295
- name: Compile
292296
working-directory: extensions/copilot
293297
run: npm run compile
@@ -383,6 +387,10 @@ jobs:
383387
working-directory: extensions/copilot
384388
run: npm run typecheck
385389

390+
- name: Lint
391+
working-directory: extensions/copilot
392+
run: npm run lint
393+
386394
- name: Compile
387395
working-directory: extensions/copilot
388396
run: npm run compile

build/filters.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -233,8 +233,10 @@ export const tsFormattingFilter = Object.freeze<string[]>([
233233
]);
234234

235235
export const eslintFilter = Object.freeze<string[]>([
236-
'**/*.{js,cjs,mjs}',
237-
'**/*.{ts,tsx,mts,cts}',
236+
'**/*.js',
237+
'**/*.cjs',
238+
'**/*.mjs',
239+
'**/*.ts',
238240
'.eslint-plugin-local/**/*.ts',
239241
...readFileSync(join(import.meta.dirname, '..', '.eslint-ignore'))
240242
.toString()

build/hygiene.ts

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -314,6 +314,20 @@ if (import.meta.main) {
314314
}
315315
}
316316

317+
// Run copilot pre-commit checks if copilot files are staged
318+
if (some.some(f => f.startsWith('extensions/copilot/'))) {
319+
console.log('Running copilot pre-commit checks...');
320+
const result = cp.spawnSync('npx', ['lint-staged'], {
321+
cwd: path.join(process.cwd(), 'extensions', 'copilot'),
322+
stdio: 'inherit',
323+
shell: true,
324+
});
325+
if (result.status !== 0) {
326+
console.error('Copilot pre-commit checks failed.');
327+
process.exit(1);
328+
}
329+
}
330+
317331
console.log('Reading git index versions...');
318332

319333
createGitIndexVinyls(some)

0 commit comments

Comments
 (0)