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 24 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 29017c5 to 6610aee Compare April 23, 2025 11:14
@rimi-itk rimi-itk force-pushed the feature/github-actions-templates branch from fb0554f to e4023ec Compare May 2, 2025 09:17
@rimi-itk rimi-itk force-pushed the feature/github-actions-templates branch from e4023ec to fb1efe1 Compare May 2, 2025 09:25
@rimi-itk rimi-itk force-pushed the feature/github-actions-templates branch 2 times, most recently 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
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