Skip to content
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

1.2.1 proposed #115

Merged
merged 129 commits into from
Jan 3, 2024
Merged

1.2.1 proposed #115

merged 129 commits into from
Jan 3, 2024

Conversation

mr-c
Copy link
Member

@mr-c mr-c commented Sep 2, 2021

A standing PR to see all the CWL v1.2.1 proposals that have gotten initial approval

  • CommandLineTool changelog for v1.2.1
  • Workflow / ExpressionTool / Operation changelog for v1.2.1

https://deploy-preview-115--cwl-v1-2-dev.netlify.app/workflow#Changelog_for_v1.2.1

https://deploy-preview-115--cwl-v1-2-dev.netlify.app/commandlinetool#Changelog_for_v1.2.1

mr-c and others added 12 commits October 9, 2021 10:13
* some defaults should not be set using 'default'

To implement the slightly complicated default value rules for
ResourceRequirement

* InputBinding.position default is 0.

* File.streamable default is false.
also run the tests against cwltool
Changes it to reference another input parameter instead of using self.
The behavior of processing secondary files patterns in order and being
able to reference earlier ones later is not part of the spec.

This doesn't exactly replicate the previous behavior, because it
introduces a new input parameter, however it does demonstrate the
ability to rename a file and have it staged as a secondary file
without having to use InitialWorkDir.

refs #127

Co-authored-by: Michael R. Crusoe <1330696+mr-c@users.noreply.github.com>
@GlassOfWhiskey
Copy link
Collaborator

StreamFlow passes all new Conformance tests. Nothing alarming in the CHANGELOG. For me, we can proceed.

mr-c and others added 4 commits September 7, 2023 12:37
Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [rojopolis/spellcheck-github-actions](https://github.com/rojopolis/spellcheck-github-actions) from 0.33.1 to 0.34.0.
- [Release notes](https://github.com/rojopolis/spellcheck-github-actions/releases)
- [Changelog](https://github.com/rojopolis/spellcheck-github-actions/blob/master/CHANGELOG.md)
- [Commits](rojopolis/spellcheck-github-actions@0.33.1...0.34.0)

---
updated-dependencies:
- dependency-name: rojopolis/spellcheck-github-actions
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
CommandLineTool.yml Outdated Show resolved Hide resolved
CommandLineTool.yml Outdated Show resolved Hide resolved
CommandLineTool.yml Outdated Show resolved Hide resolved
CommandLineTool.yml Outdated Show resolved Hide resolved
CommandLineTool.yml Outdated Show resolved Hide resolved
Workflow.yml Outdated Show resolved Hide resolved
Co-authored-by: Peter Amstutz <peter.amstutz@curii.com>
@mr-c mr-c requested a review from tetron October 30, 2023 20:10
@fmigneault
Copy link
Contributor

Closes crim-ca/weaver#549

tetron
tetron previously approved these changes Nov 6, 2023
@stain
Copy link
Member

stain commented Nov 7, 2023

All good from me 👍

@cwl-bot
Copy link

cwl-bot commented Nov 10, 2023

This pull request has been mentioned on Common Workflow Language Discourse. There might be relevant details there:

https://cwl.discourse.group/t/cwl-v1-2-1-is-coming-soon/839/1

@manabuishii
Copy link

Looks Good For Me

@tom-tan
Copy link
Member

tom-tan commented Nov 12, 2023

+1!

@mr-c
Copy link
Member Author

mr-c commented Nov 14, 2023

Of the 10 members of the CWL Technical Team I see approvals from the following:

  • Myself, Michael R. Crusoe
  • Lon Blauvelt
  • Peter Amstutz
  • Stian Soiland-Reyes
  • Tomoya Tanjo
  • Manabu Ishii
  • Marius van den Beek
  • Iacopo Colonnelli

It isn't clear to me if Fabrice Brito has approved this.

That's 7 8 approvals out of the 10 total.

Assuming we don't hear any objection from @bogdang989 @fabricebrito @GlassOfWhiskey , I now ask @tetron to send this to the CWL Leadership team for final approval.

@GlassOfWhiskey
Copy link
Collaborator

Sorry for the delay. LGTM

Bumps [rojopolis/spellcheck-github-actions](https://github.com/rojopolis/spellcheck-github-actions) from 0.34.0 to 0.35.0.
- [Release notes](https://github.com/rojopolis/spellcheck-github-actions/releases)
- [Changelog](https://github.com/rojopolis/spellcheck-github-actions/blob/master/CHANGELOG.md)
- [Commits](rojopolis/spellcheck-github-actions@0.34.0...0.35.0)

---
updated-dependencies:
- dependency-name: rojopolis/spellcheck-github-actions
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@fabricebrito
Copy link

@mr-c working on calrissian conformance tests this week. Will keep you posted.

Bumps [actions/setup-python](https://github.com/actions/setup-python) from 4 to 5.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](actions/setup-python@v4...v5)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@mr-c mr-c merged commit ae6899d into main Jan 3, 2024
18 checks passed
@mr-c mr-c deleted the 1.2.1_proposed branch April 5, 2024 08:24
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.