Skip to content

Conversation

@paulzierep
Copy link
Contributor

FOR CONTRIBUTOR:

  • I have read the Adding workflows guidelines
  • License permits unrestricted use (educational + commercial)
  • Please also take note of the reviewer guidelines below to facilitate a smooth review process.

Workflow updates see Changelog

@paulzierep paulzierep changed the title Update workflow to 0.4 Update MAGs workflow to 0.4 Oct 7, 2025
Copy link
Member

@wm75 wm75 left a comment

Choose a reason for hiding this comment

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

Apart from my one comment this is looking good.

Would be nice if you could also provide readme and help in the workflow.

@wm75
Copy link
Member

wm75 commented Oct 7, 2025

one small cosmetic thing: we usually put the release near the top of the workflow between license and wf name.

@paulzierep
Copy link
Contributor Author

@wm75 good ?
ping @AnnaSyme this version for further updates. If you want you can fork my repo and make a PR with changes to this branch.

@AnnaSyme
Copy link
Contributor

AnnaSyme commented Oct 9, 2025

Thanks @paulzierep - I'll just send you the link to the workflow (in slack) so you can check it looks ok.

@paulzierep
Copy link
Contributor Author

paulzierep commented Nov 11, 2025

@wm75 can we merge this one ? or @mvdbeek ?

@paulzierep
Copy link
Contributor Author

@mvdbeek the mentioned subworkflow is no longer part of the WF. The only subworkflow is the mapping over workflow of metaspedes, and there it makes sense imo since both tools are part of one conditional.
Changelog was adjusted.

@paulzierep paulzierep requested a review from mvdbeek November 21, 2025 13:11
@mvdbeek mvdbeek merged commit cd7f173 into galaxyproject:main Nov 21, 2025
7 checks passed
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.

4 participants