Skip to content

Commit

Permalink
Merge pull request #3685 from github/dependabot/npm_and_yarn/extensio…
Browse files Browse the repository at this point in the history
…ns/ql-vscode/typescript-eslint-7954a73ad2

Bump the typescript-eslint group across 1 directory with 2 updates
  • Loading branch information
robertbrignull authored Aug 14, 2024
2 parents 8326ef3 + cddf078 commit 14c974e
Show file tree
Hide file tree
Showing 32 changed files with 452 additions and 238 deletions.
9 changes: 9 additions & 0 deletions extensions/ql-vscode/.eslintignore
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,14 @@ node_modules/
out/
build/

# Ignore js files
.eslintrc.js
jest.config.js
test/vscode-tests/activated-extension/jest-runner-vscode.config.js
test/vscode-tests/cli-integration/jest-runner-vscode.config.js
test/vscode-tests/jest-runner-vscode.config.base.js
test/vscode-tests/minimal-workspace/jest-runner-vscode.config.js
test/vscode-tests/no-workspace/jest-runner-vscode.config.js

# Include the Storybook config
!.storybook
15 changes: 2 additions & 13 deletions extensions/ql-vscode/.eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ const baseConfig = {
"@typescript-eslint/no-invalid-this": "off",
"@typescript-eslint/no-shadow": "off",
"prefer-const": ["warn", { destructuring: "all" }],
"@typescript-eslint/no-throw-literal": "error",
"@typescript-eslint/only-throw-error": "error",
"@typescript-eslint/consistent-type-imports": "error",
"import/consistent-type-specifier-style": ["error", "prefer-top-level"],
curly: ["error", "all"],
Expand Down Expand Up @@ -133,18 +133,7 @@ module.exports = {
...baseConfig.rules,
// We want to allow mocking of functions in modules, so we need to allow namespace imports.
"import/no-namespace": "off",
"@typescript-eslint/ban-types": [
"error",
{
// For a full list of the default banned types, see:
// https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/ban-types.md
extendDefaults: true,
types: {
// Don't complain about the `Function` type in test files. (Default is `true`.)
Function: false,
},
},
],
"@typescript-eslint/no-unsafe-function-type": "off",
},
},
{
Expand Down
2 changes: 1 addition & 1 deletion extensions/ql-vscode/gulpfile.ts/appInsights.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { src, dest } from "gulp";
// eslint-disable-next-line @typescript-eslint/no-var-requires,import/no-commonjs
// eslint-disable-next-line @typescript-eslint/no-require-imports,import/no-commonjs
const replace = require("gulp-replace");

/** Inject the application insights key into the telemetry file */
Expand Down
Loading

0 comments on commit 14c974e

Please sign in to comment.