noir-lang / vscode-noir

Apache License 2.0

6 stars 7 forks source link

chore: add linter setup #57

Closed TomAFrench closed 9 months ago

TomAFrench commented 9 months ago

Description

Problem*

Resolves

Summary*

This PR adds the standard linter setup from the Noir monorepo.

As part of this I noticed that we weren't handling brillig opcodes in EditorLineDecorationManager which were hidden by not properly typing the functions. This results in no brillig opcodes being reported.

Additional Context

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

socket-security[bot] commented 9 months ago

New, updated, and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Packages	Version	New capabilities	Transitives	Size	Publisher
@typescript-eslint/eslint-plugin	6.14.0	None	`+27`	10.1 MB
eslint-plugin-prettier	5.0.1	None	`+54`	6.2 MB	jounqin
@typescript-eslint/parser	5.59.9...6.14.0	None	`+22/-22`	7.11 MB
eslint	8.42.0...8.55.0	None	`+14/-13`	5.2 MB	eslintbot

🚮 Removed packages: prettier@2.8.8