-
Notifications
You must be signed in to change notification settings - Fork 29
trs extensions to facilitate toil development #6005
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
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #6005 +/- ##
=============================================
+ Coverage 74.45% 74.47% +0.02%
- Complexity 5495 5497 +2
=============================================
Files 381 381
Lines 19786 19802 +16
Branches 2043 2044 +1
=============================================
+ Hits 14731 14747 +16
Misses 4075 4075
Partials 980 980
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
Note will need to make a tag on our fork of the TRS repository on PR approval and update PR to match |
dockstore-webservice/src/main/java/io/openapi/api/impl/ToolsApiServiceImpl.java
Outdated
Show resolved
Hide resolved
|
Description
Add
dockstore_absolute_pathto files endpoint and descriptorsAdd
dockstore_self_urlto descriptorsReview Instructions
Load up a test DB and try the following urls
http://hostname:4200/api/ga4gh/trs/v2/tools/%23workflow%2Fgithub.com%2Fbroadinstitute%2Fwarp%2FOptimus/versions/develop/WDL/files will have list the files and have dockstore_absolute_path for each file in the list
http://hostname:4200/api/ga4gh/trs/v2/tools/%23workflow%2Fgithub.com%2Fbroadinstitute%2Fwarp%2FOptimus/versions/develop/WDL/descriptor will show the json for one file and have both dockstore_absolute_path and dockstore_self_url
Issue
https://ucsc-cgl.atlassian.net/browse/SEAB-6699
Security and Privacy
None
Please make sure that you've checked the following before submitting your pull request. Thanks!
mvn clean install@RolesAllowedannotation