noir-lang / vscode-noir

Apache License 2.0
6 stars 7 forks source link

feat(aztec): search for aztec-nargo on top of nargo bin #67

Closed kobyhallx closed 6 months ago

kobyhallx commented 6 months ago

Partialy resolves

LSP to set nargo path based on nargo or aztec-nargo #4233,

Adds

image

Status Bar item with information about nargo picked.

Indicates with colour and diagnostic message otherwise

image

TomAFrench commented 6 months ago

We should also make sure to handle the edge case where the user has a workspace which contains a mix of contracts and other package types. In this case we should use aztec-nargo to ensure we are using a consistent version of nargo across the entire workspace.

TomAFrench commented 6 months ago

Agreed in call that automatic handling of the different versions shouldn't be a requirement for this PR as that would prevent us from moving forwards on this until the next nargo release. Instead we're going to make the status icon clickable to switch between found versions of nargo.