tools-iuc icon indicating copy to clipboard operation
tools-iuc copied to clipboard

Make IUC workflows reusable

Open bernt-matthias opened this issue 1 year ago • 0 comments

This PR adds:

  1. Reusable Workflows which can be used in all IUC style repos
    • The hope/aim is that users of these workflows will need to update/sync their workflows less often. In particular the galaxy-branch will be updated automatically per default once its updated at IUC. Still users can fix the branch if they like. Also parameters for linting and testing are by default as they are at IUC, but can be overwritten.
    • Bugs in Galaxy (tool-util,...) and planemo will impact more repos by default (if repos do not fix the for and branch). I would argue that this is good because it might help to detect bugs early.
    • TODO it would be handy if we could do the switch from master to main, since called worflows apparently can't refer to the default branch. otherwise we would need to change the calling workflows again.
  2. Adds report-level and fail-level
    • supersedes https://github.com/galaxyproject/galaxy-tool-repository-template/pull/8
  3. Update all called actions
    • checkout v2 -> v3
    • cache v2 -> v3
    • upload-artifact v2 -> v3
    • download-artifact v2 -> v3
    • setup-python v1 -> v4
    • peter-evans/create-or-update-comment v1 -> v2
    • peter-evans/slash-command-dispatch v2 -> v3

The PR can also be seen in action here: https://github.com/galaxyproject/galaxy-tool-repository-template/pull/11

TODOs:

  • [ ] test to use from devteam / template repo (https://github.com/galaxyproject/galaxy-tool-repository-template/pull/11)

FOR CONTRIBUTOR:

  • [ ] - I have read the CONTRIBUTING.md document and this tool is appropriate for the tools-iuc repo.
  • [ ] - License permits unrestricted use (educational + commercial)
  • [ ] - This PR adds a new tool or tool collection
  • [ ] - This PR updates an existing tool or tool collection
  • [x] - This PR does something else (explain below)

bernt-matthias avatar Oct 04 '22 13:10 bernt-matthias