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.

Sign up for GitHub

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

Jump to bottom

fractional resource requirements for CWL 1.1 and 1.2 spec #230

Open
wants to merge 74 commits into
base: develop-v1.2
Choose a base branch
from
Open

fractional resource requirements for CWL 1.1 and 1.2 spec #230

wants to merge 74 commits into from

Conversation

vsmalladi
Copy link
Contributor

Closes #139

alextsaihi and others added 30 commits February 5, 2024 10:45
@alextsaihi
Update README.md
4a56260
@vsmalladi
Fix build docs.
0c95751
@vsmalladi
Update build.yml
ebde4bb 
Update versions
@vsmalladi
Update ci.yml
1de8ebe 
Updated actions versions
@vsmalladi
Merge pull request #218 from vsmalladi/197-branch
d49bc65 
docs: fix build
@vsmalladi
Merge pull request #201 from alextsaihi/ga4gh-logo-update
2082a8f 
update logo source
@vsmalladi
Fix some of the description in readme and tags. (#1)
a5fb76e 
* Fix some of the description in readme and tags.

* Fix build to run based on requirements of plugins
@vsmalladi
Resolve issue with two sections
b0c08ee
@vsmalladi
Merge pull request #220 from vsmalladi/develop
4362315 
Fi build and readme
@vsmalladi
Update logo
b0eb2d5
@vsmalladi
Merge pull request #221 from vsmalladi/develop
c157712 
Update logo to render in html documentation
@vsmalladi
Update to be linked to the latest tag.
42bb695
@vsmalladi
Fix repo.
a0da756
@vsmalladi
Add back build and spec badges.
93b1e3a
@vsmalladi
Check with develop.
0c05b27
@vsmalladi
Testing ci badge.
d04eca1
@vsmalladi
Testing ci badge.
6288dc4
@vsmalladi
check validtor.
c4f8d98
@vsmalladi
Make a table and visualize all on the status for both master and deve…
244a8b8 
…lopment.
@vsmalladi
Change the develop branch name.
4bf02c6
@vsmalladi
Reword description.
f6495ef
@vsmalladi
Fix formating and full stops.
d9ebe9c
@vsmalladi
Add rewording.
c02bcb9
@vsmalladi
Update JSON.
a248cde
@vsmalladi
Remove Possible future enhancemnts section.
b287476
@vsmalladi
Add Governance.
7f253a0
@vsmalladi
Add in license.
9b19cf7
@vsmalladi
Fix badges.
8614de7
@vsmalladi
Simplify build.
07cbe93
@vsmalladi
Build in PR.
aab9eba
uniqueg and others added 23 commits October 29, 2024 15:12
@uniqueg
Update README.md
9a38e48
@uniqueg
Update README.md
e6a5275
@uniqueg
Update README.md
5fdc081
@uniqueg
Update README.md
e87ac17
@uniqueg
Update CONTRIBUTING.md
cf7b478
@uniqueg
Update CONTRIBUTING.md
22d7ae7
@vsmalladi
Merge pull request #223 from ga4gh/uniqueg-patch-1
df497df 
Update GOVERNANCE.md
@vsmalladi
Merge pull request #224 from ga4gh/uniqueg-patch-2
6f40e15 
Update build.yml
@vsmalladi
Merge pull request #225 from ga4gh/uniqueg-patch-3
e15a1aa 
Update README.md
@vsmalladi
Merge pull request #226 from ga4gh/uniqueg-patch-4
3ba6a09 
Update CONTRIBUTING.md
@uniqueg
Create diff_comment.yml
8fed9ee
@uniqueg
Update diff_comment.yml
556b134
@uniqueg
Update diff_comment.yml
ed4499b
@uniqueg
Update diff_comment.yml
2f6a18a
@uniqueg
Update diff_comment.yml
2ea05ea
@uniqueg
Update diff_comment.yml
7cb1d0c
@uniqueg
Update diff_comment.yml
8379c63
@uniqueg
Update diff_comment.yml
98143d2
@uniqueg
Update diff_comment.yml
dc099c8
@uniqueg
Update diff_comment.yml
d41e922
@uniqueg
Update diff_comment.yml
13e7b96
@uniqueg
Delete .github/workflows/diff_comment.yml
f099807
@vsmalladi
Add in fractional values for cwl
1f58489
@vsmalladi vsmalladi added this to the 1.2 milestone Nov 8, 2024
@vsmalladi vsmalladi mentioned this pull request Nov 8, 2024
Open
uniqueg
uniqueg approved these changes Nov 10, 2024
View reviewed changes
Copy link
Contributor

@uniqueg uniqueg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but would perhaps wait with merging until #219 is merged and develop is merged into develop-v1.2

mr-c
mr-c reviewed Nov 14, 2024
View reviewed changes
openapi/task_execution_service.openapi.yaml Outdated Show resolved Hide resolved
openapi/task_execution_service.openapi.yaml Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mr-c mr-c mr-c left review comments

@uniqueg uniqueg uniqueg approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.2
Development

Successfully merging this pull request may close these issues.
4 participants
@vsmalladi @mr-c @uniqueg @alextsaihi