Skip to content

wdl-lint: add NoHostPathLiterals rule#736

Open
raman976 wants to merge 5 commits intostjude-rust-labs:mainfrom
raman976:feat/no-host-path-literals
Open

wdl-lint: add NoHostPathLiterals rule#736
raman976 wants to merge 5 commits intostjude-rust-labs:mainfrom
raman976:feat/no-host-path-literals

Conversation

@raman976
Copy link
Contributor

Fixes #718

This PR adds the NoHostPathLiterals lint rule to flag File and Directory declarations in input and private sections that use absolute host path literals as defaults. Absolute paths resolve on the host filesystem, making tasks non-portable across environments.

Before submitting this PR, please make sure:

For external contributors:

  • You have read the contributing guide in its entirety.
  • You have not used AI on any parts of this pull request.
  • You have added a few sentences describing the PR here.
  • Your code builds clean without any errors or warnings.

For all contributors:

  • You have added tests (when appropriate).
  • You have added an entry in the CHANGELOG (when appropriate).
  • You have updated the README or other documentation to account for these changes (when appropriate).
  • You have made a PR to the next branch in the sprocket.bio repository (when appropriate).

For PRs containing lint rule changes:

  • You have updated any and all effected entries within RULES.md.
  • You have added a test case in crates/wdl-lint/tests/lints that covers every
    possible diagnostic emitted for the rule within the file where the rule
    is implemented.

@raman976 raman976 requested a review from a team as a code owner March 18, 2026 05:45
@raman976 raman976 requested a review from claymcleod March 18, 2026 05:45
@github-actions github-actions bot added the S-awaiting-pass-CI PR is awaiting CI to pass. label Mar 18, 2026
@raman976
Copy link
Contributor Author

Hi @claymcleod , please review this PR and let me know if there are any changes that need to be done. I would be happy to do them.

@Serial-ATA Serial-ATA self-requested a review March 18, 2026 20:39
@github-actions github-actions bot removed the S-awaiting-pass-CI PR is awaiting CI to pass. label Mar 19, 2026
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

Successfully merging this pull request may close these issues.

New wdl-lint rule: NoHostPathLiterals

1 participant