Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@denis-yuen
Copy link
Member

@denis-yuen denis-yuen commented Dec 18, 2023

Description
Use result of dockstore/cwlavro#42

Review Instructions
Generate a JSON parameter file for a cwl workflow and a cwl tool
Launch a cwl tool and workflow

Issue
https://ucsc-cgl.atlassian.net/browse/SEAB-6017

Security
If there are any concerns that require extra attention from the security team, highlight them here.

Please make sure that you've checked the following before submitting your pull request. Thanks!

  • Check that you pass the basic style checks and unit tests by running ./mvnw clean install
  • If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.

@denis-yuen denis-yuen self-assigned this Dec 18, 2023
@denis-yuen denis-yuen changed the title awlavro upgrade cwlavro upgrade Dec 18, 2023
@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@codecov
Copy link

codecov bot commented Dec 18, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (071e320) 70.12% compared to head (50404a8) 70.14%.
Report is 2 commits behind head on develop.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop     #271      +/-   ##
=============================================
+ Coverage      70.12%   70.14%   +0.01%     
- Complexity      1060     1064       +4     
=============================================
  Files             47       47              
  Lines           6069     6069              
  Branches         801      801              
=============================================
+ Hits            4256     4257       +1     
  Misses          1466     1466              
+ Partials         347      346       -1     
Flag Coverage Δ
bitbuckettests 9.83% <ø> (ø)
confidentialtooltests 55.79% <ø> (+0.18%) ⬆️
confidentialworkflowtests 30.18% <ø> (-0.12%) ⬇️
nonconfidentialtests 32.36% <ø> (ø)
singularitytests 16.59% <ø> (ø)
unittests 8.35% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@denis-yuen denis-yuen marked this pull request as ready for review December 18, 2023 21:57
@denis-yuen denis-yuen requested review from a team, coverbeck, david4096, hyunnaye and kathy-t and removed request for a team December 18, 2023 21:58
@denis-yuen denis-yuen merged commit 455fb0d into develop Jan 2, 2024
@denis-yuen denis-yuen deleted the avro_update branch January 2, 2024 16:54
denis-yuen added a commit that referenced this pull request Jan 2, 2024
@denis-yuen denis-yuen mentioned this pull request Jan 2, 2024
2 tasks
denis-yuen added a commit that referenced this pull request Jan 3, 2024
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.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.