Skip to content

Feature/GitHub actions maintenance#140

Merged
turegjorup merged 7 commits intodevelopfrom
feature/github-actions_maintenance
Mar 23, 2026
Merged

Feature/GitHub actions maintenance#140
turegjorup merged 7 commits intodevelopfrom
feature/github-actions_maintenance

Conversation

@turegjorup
Copy link
Contributor

Ticket

7043

Summary

  • Update actions/checkout from v5 to v6 across all workflow templates (github/workflows/) and project workflows (.github/workflows/)
  • Add paths filters to file-type-specific linter workflows so they only trigger when relevant files are changed, reducing unnecessary CI runs:
    • PHP workflows: **/*.php, composer.json, composer.lock, docker-compose.yml
    • Composer workflow: composer.json, composer.lock, docker-compose.yml
    • Twig workflow: **/*.twig, composer.json, composer.lock, docker-compose.yml
    • Markdown workflow: **/*.md, docker-compose.yml
    • YAML workflow: **/.yml, **/.yaml, docker-compose.yml
    • JavaScript workflows: scoped JS paths + docker-compose.yml
    • Styles workflows: scoped CSS/SCSS paths + docker-compose.yml
  • All workflows also trigger on docker-compose.yml changes since the linter tool versions are defined by Docker image tags in compose
  • PHP and Twig workflows also trigger on composer.json/composer.lock changes since linter packages are managed via Composer
  • Remove redundant fail-fast: false from jobs without a matrix strategy (changelog.yaml, composer.yaml)

@turegjorup turegjorup self-assigned this Mar 23, 2026
@turegjorup turegjorup requested review from cableman and rimi-itk March 23, 2026 11:50
@turegjorup turegjorup mentioned this pull request Mar 23, 2026
Copy link
Contributor

@rimi-itk rimi-itk left a comment

Choose a reason for hiding this comment

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

Looks good. Using a YAML anchor for paths would make this a little easier to maintain.

paths:
- "**/*.md"
- "compose.{yaml,yml}"
- "docker-compose.{yaml,yml}"
Copy link
Contributor

Choose a reason for hiding this comment

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

Consider using a YAML anchor and reference for the paths, e.g.

    paths: &paths
      - "**/*.md"
      - "compose.{yaml,yml}"
      - "docker-compose.{yaml,yml}"
    
    # …
      
  push:
    branches:
      - main
      - develop
    paths: *paths

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nice improvement. Keep forgetting anchors are supported now.

@turegjorup turegjorup requested a review from rimi-itk March 23, 2026 12:36
@turegjorup turegjorup merged commit 0bc5a96 into develop Mar 23, 2026
5 checks passed
@turegjorup turegjorup deleted the feature/github-actions_maintenance branch March 23, 2026 12:43
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