Skip to content

Try to fix planemo test workflow when output is collection with identical name#1391

Merged
mvdbeek merged 2 commits intogalaxyproject:masterfrom
lldelisle:wf_same_output_name
Oct 6, 2023
Merged

Try to fix planemo test workflow when output is collection with identical name#1391
mvdbeek merged 2 commits intogalaxyproject:masterfrom
lldelisle:wf_same_output_name

Conversation

@lldelisle
Copy link

Lucille Delisle and others added 2 commits October 5, 2023 15:37
Don't love it but it'll do for now.
@lldelisle
Copy link
Author

lldelisle commented Oct 6, 2023

Any idea why the test is not failing on CI?
What did I do wrong? Because it was failing locally...

@mvdbeek
Copy link
Member

mvdbeek commented Oct 6, 2023

It would have run in https://github.com/galaxyproject/planemo/actions/runs/6421040087/job/17434490532, but that was cancelled.

@mvdbeek
Copy link
Member

mvdbeek commented Oct 6, 2023

And yeah, it's failing locally for me too and my commit fixes this.

@lldelisle
Copy link
Author

Great!
Should I try to fix the CI into another PR?

@mvdbeek
Copy link
Member

mvdbeek commented Oct 6, 2023

I think this is conda timing out, that probably needs to be fixed on the Galaxy or Conda side ...

@mvdbeek mvdbeek merged commit 9daa9ba into galaxyproject:master Oct 6, 2023
@lldelisle lldelisle deleted the wf_same_output_name branch October 6, 2023 18:57
@lldelisle
Copy link
Author

Thanks for the fix.

@mvdbeek
Copy link
Member

mvdbeek commented Oct 6, 2023

Thanks for the test, that made it easy!

mvdbeek added a commit to mvdbeek/planemo that referenced this pull request Oct 26, 2023
The previous checksum is for an empty string. I think this has been
wrong from the start because the code for downloading outputs was broken
and only fixed in galaxyproject#1391
mvdbeek added a commit to mvdbeek/planemo that referenced this pull request Oct 26, 2023
The previous checksum is for an empty string. I think this has been
wrong from the start because the code for downloading outputs was broken
and only fixed in galaxyproject#1391
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.

2 participants