By participating in this project, you agree to abide by our Code of Conduct. Please read it before contributing.
New unit test suggestions are tracked as GitHub issues. When creating a unit test suggestion, please:
- First check the issue tracker to identify if your issue/proposed unit test is already in development. This avoids duplication and promotes collaborative efforts on problems of common interest.
- Use a clear and descriptive title
- Provide a detailed description of what the unit test is actually intending to test?
- Explain why this additional test would be useful
- Once you have identified a new unit test and would like to contribute towards building this repository, begin by filing an issue in this repository.
- Clone the repo locally and create a branch from
main
. Please name the branch using this convention: "issue#-issue-title" - Create a very specific WDL unit test and make sure to add enough commentary in the code.
- Ensure your proposed WDL runs locally via
miniwdl run
orjava -jar cromwell-86.jar run
- If you are user of PROOF, please also make sure to test that the WDL unit test runs via PROOF.
- If your WDL succeeds locally but fails in PROOF, please report the issue in the proof-api repo.
- Add the WDL to a subdirectory by the same name, i.e.
coolUnitTest/coolUnitTest.wdl
. If the WDL is expected to fail WOMtool the subdirectory AND the WDL file name must start withbadVal
. If the WDL is expected to fail both WOMtool and a Cromwell run, then the subdirectory AND the WDL file name must start withbadRun
. - Make sure to include an
inputs.json
andoptions.json
if required and make sure that any other. input files referenced ininputs.json
are provided in the same directory. - Add a README to the WDL's subdirectory describing the unit test's functionality and purpose.
- Commit and push your proposed changes to GitHub.
- Create a pull request describing the updates and identifying the corresponding GitHub issue.
- Request a review from a member of the DaSL team (ideally the person requesting the test).
- Address all comments/requested changes from these reviews.
- Once the PR has been approved and all checks have passed, merge the PR into
main
.
- Follow WILDS WDL Style Guide
- For now, only use WDL v1.0 syntax (sadly Cromwell only accepts v1.0)
By contributing, you agree that your contributions will be licensed under the same license as the original project.