Commit dd1f805

committed

feat(shellcheck): add code actions to disable rule

Implements #933. Always add a code action to disable a given diagnostic for a given line or for the whole file. This patch handles: * indentation * existing directive. It also sorts the disable directives if any * multi-line command Add corresponding tests. Signed-off-by: Thomas Faivre <thfaivre@gmail.com>

1 parent 29621ff commit dd1f805Copy full SHA for dd1f805

File tree

4 files changed

+2274

-69

lines changed

server/src
- __tests__
  - server.test.ts
- server.ts
- shellcheck
  - __tests__
    - index.test.ts
  - index.ts

4 files changed

+2274

-69

lines changed

`‎server/src/tests/server.test.ts`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -222,7 +222,7 @@ describe('server', () => {`
`222`	`222`	`{} as any,`
`223`	`223`	`)`
`224`	`224`
`225`		`- expect(result).toHaveLength(1)`
	`225`	`+ expect(result).toHaveLength(3)`
`226`	`226`	`const codeAction = (result as CodeAction[])[0]`
`227`	`227`	`expect(codeAction.diagnostics).toEqual([fixableDiagnostic])`
`228`	`228`	`expect(codeAction.diagnostics).toEqual([fixableDiagnostic])`

`‎server/src/server.ts`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -405,7 +405,7 @@ export default class BashServer {`
`405`	`405`
`406`	`406`	`const codeActions = params.context.diagnostics`
`407`	`407`	`.map(({ data }) => codeActionsForUri[data?.id])`
`408`		`- .filter((action): action is LSP.CodeAction=>action!=null)`
	`408`	`+ .flat()`
`409`	`409`
`410`	`410`	logger.debug(`onCodeAction: found ${codeActions.length} code action(s)`)
`411`	`411`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit dd1f805

File tree

4 files changed

4 files changed

`‎server/src/tests/server.test.ts`

`‎server/src/server.ts`

0 commit comments

File tree

4 files changed

4 files changed

‎server/src/__tests__/server.test.ts

‎server/src/server.ts

0 commit comments

`‎server/src/tests/server.test.ts`

`‎server/src/server.ts`