Conversation
There was a problem hiding this comment.
Pull request overview
This pull request updates the repository’s CI workflow to improve build speed/reliability (via concurrency + Docker Buildx caching + Composer download caching) and removes the app_events.md documentation file from the repo.
Changes:
- Add workflow-level concurrency to auto-cancel superseded runs on the same ref.
- Switch dev image build to Docker Buildx with GitHub Actions cache backend, and add Composer download caching mounted into the container.
- Remove
app_events.md.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
.github/workflows/ci.yml |
Adds concurrency + permissions, builds the dev image with buildx+GHA cache, and caches Composer downloads for faster installs. |
app_events.md |
Removes the application events documentation file. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
arkdevuk
added a commit
that referenced
this pull request
Jul 20, 2026
Merge pull request #1 from arkdevuk/dev
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request updates the CI workflow and removes the
app_events.mddocumentation file. The CI workflow changes focus on improving build performance and reliability by adding concurrency controls, using Docker Buildx with caching, and caching Composer downloads. The removal ofapp_events.mdeliminates the documentation of application-level events.CI Workflow Improvements:
concurrencygroup to the workflow to auto-cancel stale runs for the same branch or PR, preventing wasted CI minutes.permissionsfor the workflow, restricting it to read-only access to repository contents for increased security.make buildstep with Docker Buildx for building the dev image, enabling GitHub Actions cache for faster builds and avoiding unnecessary rebuilds.actions/cacheaction and mounted the cache into the container, speeding up dependency installation. [1] [2]COMPOSER_CACHE_DIRenvironment variable, further optimizing dependency management.Documentation Removal:
app_events.mdfile, which previously documented all application-level events, their payloads, and dispatch triggers.