Skip to content

Cache Box PHAR in reusable binary workflow #206

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.

Sign up for GitHub

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
rahimahisah17 wants to merge 2 commits into
base: develop
Choose a base branch
from
+36 −3
Open

Cache Box PHAR in reusable binary workflow #206

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1cf5ae3
Cache Box PHAR in reusable binary workflow
rahimahisah17 Sep 3, 2026
1483ce1
Address review feedback for Box cache
rahimahisah17 Sep 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .github/workflows/release.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,8 @@ jobs:

name: Build binary
uses: ./.github/workflows/reusable-build-binary.yml
with:
use-cache: false

verify:
name: Validate binary on PHP ${{ matrix.php }}
Expand Down
37 changes: 34 additions & 3 deletions .github/workflows/reusable-build-binary.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,12 @@ name: Build Binary

on:
workflow_call:
inputs:
use-cache:
description: "Whether to use the Box PHAR cache"
required: false
type: boolean
default: true

jobs:
bundle:
Expand Down Expand Up @@ -31,11 +37,36 @@ jobs:
# Bust the cache at least once a month - output format: YYYY-MM.
custom-cache-suffix: $(date -u "+%Y-%m")

# Note: do NOT turn on the requirement checker in the box config as it is no longer
# compatible with PHP < 7.2.
- name: Get latest Box version
id: box-version
run: |
latest_url=$(curl -Ls -o /dev/null -w '%{url_effective}' https://github.com/box-project/box/releases/latest)
echo "version=${latest_url##*/}" >> "$GITHUB_OUTPUT"
Comment on lines +40 to +44

@jrfnl jrfnl Sep 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I realize this mirrors the current version download of the latest release, but I wonder if we should start to take into account that the latest release may not always be the semantically latest release. This is typically something which would/could happen if a project releases for multiple majors.

Not sure whether that is applicable here at this time, but whether it is or isn't, isn't even that relevant as it could still happen in the future and if it does, we don't want to get bitten by it.

We may also want to consider limiting to the "latest of the current major", as if Box releases a new major, the configuration we use may need updates etc.

This action runner might be helpful to solve the above feedback:
https://github.com/oprypin/find-latest-tag

I've used it successfully in another repo.

Note: if the above suggestions get implemented, the "Install box" step will also need to be updated to download the PHAR based on the version number, rather than on latest.


- name: Cache Box PHAR
id: cache-box
if: inputs.use-cache
uses: actions/cache/restore@v4
with:
path: box.phar
key: box-phar-${{ steps.box-version.outputs.version }}

- name: Install Box
run: wget https://github.com/box-project/box/releases/latest/download/box.phar -O box.phar && chmod 0755 box.phar && pwd
if: ${{ !inputs.use-cache || steps.cache-box.outputs.cache-hit != 'true' }}
run: wget https://github.com/box-project/box/releases/latest/download/box.phar -O box.phar

- name: Save Box PHAR cache
if: inputs.use-cache && steps.cache-box.outputs.cache-hit != 'true'
uses: actions/cache/save@v4
with:
path: box.phar
key: box-phar-${{ steps.box-version.outputs.version }}

- name: Set Box permissions
run: chmod 0755 box.phar

# Note: do NOT turn on the requirement checker in the box config as it is no longer
# compatible with PHP < 7.2.
- name: Validate configuration
run: php box.phar validate -i box.json

Expand Down
Loading