wdl

wdl copied to clipboard

This pull request adds a new rule to `wdl`. - **Rule Name**: `document_preamble` - **Rule Kind**: Lint warning - **Rule Code**: `v1::W009` - **Packages**: `wdl-grammar` Implements document_preamble as defined in...

adthrasher

Error parsing ternary operator assignment

4

Hi folks, I think I found a subtle bug in parsing a particular type of ternary operator assignment. Here are the steps to reproduce: 1. Make the following file called...

rickymagner

Fix how exceptions work

1

In #130, we discovered a bug in how excepting rules work. Here's the description of the problem as posted in Slack and also the suggested fix: > The rule actually...

claymcleod

feat: format

1

_Describe the problem or feature in addition to a link to the issues._ New crate, `wdl-format`! Handles all things formatting! In the interest of keeping this PR from sprawling further,...

a-frantz

[conventional commit]: https://www.conventionalcommits.org/en/v1.0.0/#summary ["keep a changelog"]: https://keepachangelog.com/en/1.0.0/

a-frantz

https://github.com/stjude-rust-labs/wdl/pull/142/files#r1700936038

claymcleod

Discussed in #146, we should consider replacing all of the logging with `tracing`.

claymcleod

Add `description` requirement to `meta` requirement

2

https://github.com/stjude-rust-labs/wdl/blob/cf378aff7dba902e6b970dcd13ea746ae294e0ec/wdl-lint/src/rules/missing_metas.rs#L57 This rule should notify users that the newly added `meta` section must include a `description` key or the user will immediately receive a new lint warning from `DescriptionMissing`.

adthrasher

Configuration for where outputs should be documented

4

Continuation/migration of https://github.com/stjude-rust-labs/sprocket/issues/3 The gist: The community is divided here. Some of us are of the opinion that outputs should be documented in the `meta` section and some of us...

a-frantz

New parser implementation should accept numbers for the `default` placeholder option.

2

See discussion [here](https://github.com/stjude-rust-labs/wdl/pull/76#discussion_r1636478190). While I am of the opinion that only a string value should be accepted for the `default` placeholder option, an official parser [does accept numbers](https://github.com/openwdl/wdl-parsers/blob/main/antlr4/1.1/WdlV1_1Parser.g4#L50) as well....

peterhuene