Skip to content

Some fixes for workflow_job_init and workflow_test_init#1101

Merged
jmchilton merged 2 commits intogalaxyproject:masterfrom
simonbray:workflow-init
Dec 11, 2020
Merged

Some fixes for workflow_job_init and workflow_test_init#1101
jmchilton merged 2 commits intogalaxyproject:masterfrom
simonbray:workflow-init

Conversation

@simonbray
Copy link
Member

@simonbray simonbray changed the title Some fixes for workflow_init Some fixes for workflow_job_init and workflow_test_init Dec 3, 2020
@bgruening
Copy link
Member

Can you please add a test, so we do not regress here anymore? Thanks.

@simonbray
Copy link
Member Author

Can you please add a test, so we do not regress here anymore? Thanks.

There is a test already, but you are right, I need to make it stricter.

Also, these commands don't work yet with workflow IDs, it would be good to get that working.

@simonbray simonbray changed the title Some fixes for workflow_job_init and workflow_test_init [WIP] Some fixes for workflow_job_init and workflow_test_init Dec 5, 2020
@simonbray
Copy link
Member Author

Also, these commands don't work yet with workflow IDs, it would be good to get that working.

I will leave this for now.

@simonbray simonbray changed the title [WIP] Some fixes for workflow_job_init and workflow_test_init Some fixes for workflow_job_init and workflow_test_init Dec 8, 2020
@bgruening bgruening requested a review from jmchilton December 11, 2020 14:31
@jmchilton jmchilton merged commit 828fd6d into galaxyproject:master Dec 11, 2020
@jmchilton
Copy link
Member

Sorry for missing this! Feel free to ping me on these Planemo PRs for potentially quicker turn around.

@simonbray simonbray deleted the workflow-init branch December 11, 2020 14:37
@simonbray
Copy link
Member Author

No worries, thanks for the merge!

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