Skip to content

Conversation

@GlassOfWhiskey
Copy link
Collaborator

To prepare the library for the new ScatterWorkflowStep and LoopWorkflowStep objects introduced in CWL v1.3, this commit adds type aliases to be used in previous CWL versions.

The ScatterWorkflowStep is an alias for WorkflowStep in CWL up to v1.2, as the WorkflowStep object has the scatter and scatterMethod properties.

The LoopWorkflowStep does not exist before CWL 1.3. Therefore, a NoType abstract class has been introduced as a placeholder.

To prepare the library for the new `ScatterWorkflowStep` and
`LoopWorkflowStep` objects introduced in CWL v1.3, this commit
adds type aliases to be used in previous CWL versions.

The `ScatterWorkflowStep` is an alias for `WorkflowStep` in CWL up to
v1.2, as the `WorkflowStep` object has the `scatter` and `scatterMethod`
properties.

The `LoopWorkflowStep` does not exist before CWL 1.3. Therefore, a
`NoType` abstract class has been introduced as a placeholder.
@codecov
Copy link

codecov bot commented Sep 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 34.56%. Comparing base (fa7b09b) to head (26d07aa).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #371      +/-   ##
==========================================
+ Coverage   34.55%   34.56%   +0.01%     
==========================================
  Files          30       30              
  Lines       35375    35382       +7     
  Branches     9555     9555              
==========================================
+ Hits        12224    12231       +7     
  Misses      20266    20266              
  Partials     2885     2885              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@GlassOfWhiskey GlassOfWhiskey requested a review from mr-c September 9, 2025 20:06
@mr-c mr-c merged commit c8fba4d into main Sep 9, 2025
32 checks passed
@mr-c mr-c deleted the add-scatter-workflow-step branch September 9, 2025 20:18
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.

3 participants