search

stjude-rust-labs / wdl

Rust crates for working with Workflow Description Language (WDL) documents.
Apache License 2.0
26 stars 6 forks source link

Shorten spans highlighted by lints #243

Open a-frantz opened 5 hours ago

a-frantz commented 5 hours ago

We've had users complain that the number of highlights in the VSCode extension is overwhelming. Some of our highlights encompass too much and need to be more targeted. This should reduce the visual load in the IDE.

We should audit all our existing lint rules to make sure the spans make sense. Off the top of my head, I know the trailing comma rule will highlight the entire object that is missing a comma. This should be changed to just highlight the character immediately preceding the expected comma.

a-frantz commented 5 hours ago

This should probably be tackled in the same PR that closes #193. Similar objectives.