Skip to content

cwljava: refresh with latest schema-salad & proposed CWL v1.2.1 changes - #45

Open
mr-c wants to merge 2 commits into
dnanexus:developfrom
mr-c:cwljava_update_20220901
Open

cwljava: refresh with latest schema-salad & proposed CWL v1.2.1 changes#45
mr-c wants to merge 2 commits into
dnanexus:developfrom
mr-c:cwljava_update_20220901

Conversation

@mr-c

@mr-c mr-c commented Sep 1, 2022

Copy link
Copy Markdown
Contributor
  • SavableSaveable typo fix changes some class names
  • default field type is now explicitly including File and Directory alongside Any
  • hints is now defined as any of the known requirements, or Any (should be a nice simplification)
  • the refscope for workflow outputSource was corrected from 0 to 1

@mr-c
mr-c force-pushed the cwljava_update_20220901 branch from 63b5023 to 79064d8 Compare September 1, 2022 10:38
@mr-c

mr-c commented Sep 1, 2022

Copy link
Copy Markdown
Contributor Author

Okay, so common-workflow-language/cwljava@476b0c0 is a breaking change ; but a necessary change.

Can I get some help fixing things here in scala?

I checked the "allow edits by maintainers"; or feel free to make a PR to this PR, or your own PR directly

@mr-c

mr-c commented Jul 19, 2024

Copy link
Copy Markdown
Contributor Author

Ping :-)

It would be nice to get this merged now, so when CWL v1.3 comes (which is under development and features workflow-level loops), then the upgrade will be easier

mr-c added 2 commits July 19, 2024 07:48
- `Savable` → `Saveable` typo fix changes some class names
- `default` field type is now explicitly including `File` and `Directory` alongside `Any`
- `hints` is now defined as any of the known requirements, or `Any` (should be a nice simplification)
- the refscope for workflow `outputSource` was corrected from 0 to 1
@mr-c
mr-c force-pushed the cwljava_update_20220901 branch from 79064d8 to 335d3d5 Compare July 19, 2024 11:48
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.

1 participant