Skip to content
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

Use path.extension:row:column syntax when reporting locations #721

Open
multimeric opened this issue Dec 2, 2024 · 0 comments
Open

Use path.extension:row:column syntax when reporting locations #721

multimeric opened this issue Dec 2, 2024 · 0 comments

Comments

@multimeric
Copy link
Contributor

Currently miniWDL errors look like this:

(/path/to/file.wdl Ln 190 Col 34) Unknown identifier command_mem

However it would be advantageous to use a more machine readable format. Many IDEs inherently understand path.extension:row:column or path.extension:row, and when you click on such a string, it will automatically open the file and jump to the correct line in the editor. I believe VS Code, IntelliJ, and Sublime Text all support this.

Could we change the default error message to be like this:

(/path/to/file.wdl:190:34) Unknown identifier command_mem

Alternatively, could there be a flag passed to miniwdl check that turns on this format?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant