Skip to content

Added GitHub Actions templates and tool configuration files #104

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.

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

Open
wants to merge 27 commits into
base: develop
Choose a base branch
from

Conversation

rimi-itk
Copy link
Contributor

@rimi-itk rimi-itk commented Mar 18, 2025

https://leantime.itkdev.dk/TimeTable/TimeTable?showTicketModal=4062#/tickets/showTicket/4062

Note

This is still work in progress and currently an experiment to learn how this can be done.

Adds templates for GitHub Actions workflows that are copied to projects when installing and updating templates. The templates sit in the github/workflows/ folder and are symlink'ed into project templates. A small change has been made in itkdev-docker-compose to create actual files rather than symlinks when installing templates in a project.

Template configuration files for tools used in actions sit in the config folder.

See docs/girhub-actions-templates.md for details on what's actually going on.

To dos

  • Add a itkdev-docker-compose template:update to make it easy to update the templates in a project (based on ITKDEV_TEMPLATE=drupal-10, say, in .env, say). Command added in Added template:update command #106.
  • Be consistent on whether (or not) to use docker compose run ogrdocker run in actions
  • Add a .prettierignore

@rimi-itk rimi-itk force-pushed the feature/github-actions-templates branch 4 times, most recently from db03311 to d381d1a Compare March 18, 2025 12:53
Copy link

@sinejespersen sinejespersen left a comment

Choose a reason for hiding this comment

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

with comments

@rimi-itk rimi-itk force-pushed the feature/github-actions-templates branch 3 times, most recently from f50b20c to a6228d5 Compare March 24, 2025 11:16
@rimi-itk rimi-itk force-pushed the feature/github-actions-templates branch 14 times, most recently from ceb3413 to 8985de3 Compare April 8, 2025 08:02
@rimi-itk rimi-itk force-pushed the feature/github-actions-templates branch from ded630d to 49008d8 Compare May 2, 2025 11:50
@rimi-itk rimi-itk force-pushed the feature/github-actions-templates branch from 49008d8 to 12386c3 Compare May 2, 2025 11:52
desc: "Lint shell scripts"
cmds:
# https://github.com/koalaman/shellcheck?tab=readme-ov-file#installing
- docker run --rm --volume "$PWD:"/mnt koalaman/shellcheck:stable {{.GLOB}}

Choose a reason for hiding this comment

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

What about good old peterdavehello/shellcheck https://github.com/PeterDaveHello/docker-shellcheck

Copy link
Contributor Author

Choose a reason for hiding this comment

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

https://hub.docker.com/r/koalaman/shellcheck “is the official Docker image for ShellCheck⁠”.

@rimi-itk rimi-itk force-pushed the feature/github-actions-templates branch from 3b69e0d to 1093f59 Compare May 5, 2025 12:35
@rimi-itk rimi-itk force-pushed the feature/github-actions-templates branch from f604ef4 to 40e61eb Compare May 6, 2025 08:24
@rimi-itk rimi-itk force-pushed the feature/github-actions-templates branch from 40e61eb to b53a8fc Compare May 6, 2025 08:26
@rimi-itk rimi-itk requested review from ghbm-itk and removed request for lilosti May 6, 2025 08:28
@rimi-itk rimi-itk force-pushed the feature/github-actions-templates branch from 5a8f520 to 85aeed6 Compare May 7, 2025 11:09
@rimi-itk rimi-itk force-pushed the feature/github-actions-templates branch from 85aeed6 to 042a574 Compare May 7, 2025 11:50
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