Skip to content

expose wdl-format in wdl-lint #382

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
a-frantz opened this issue Apr 1, 2025 · 0 comments
Open

expose wdl-format in wdl-lint #382

a-frantz opened this issue Apr 1, 2025 · 0 comments
Labels
S-blocked State: blocked on an external change or decision

Comments

@a-frantz
Copy link
Member

a-frantz commented Apr 1, 2025

this is a parent issue for #379 and #380

The primary goal for this is to expose better fix messages by formatting the relevant AST nodes and outputting them in lint diagnostics. A secondary perk is that we may be able to format a node and compare it to the input node and detect possible issues that should be reported that way.

As part of this, the wdl-format API probably needs to be modified. The current implementation lends itself to only formatting specific nodes and their children in the AST, but that isn't really exposed, as so far we've only been concerned with formatting entire documents.

This is blocked for now. A higher priority is exposing wdl-analysis in wdl-lint (issue #273 and PR #341 )

@a-frantz a-frantz added the S-blocked State: blocked on an external change or decision label Apr 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-blocked State: blocked on an external change or decision
Projects
None yet
Development

No branches or pull requests

1 participant