Skip to content

enh: Make laravel.mk extensible and add multi-arch builds#1

Merged
nfebe merged 1 commit into
mainfrom
enhance/extensible-makefile-multiarch
Mar 8, 2026
Merged

enh: Make laravel.mk extensible and add multi-arch builds#1
nfebe merged 1 commit into
mainfrom
enhance/extensible-makefile-multiarch

Conversation

@nfebe

@nfebe nfebe commented Mar 8, 2026

Copy link
Copy Markdown
Contributor

Added hook variables (EXTRA_PORTS_SCRIPT, EXTRA_UP_INFO, TEST_CMD, LINT_CMD) so projects can customize behavior via include without overriding targets. Fixed DC variable to strip comments from .ports file and help target to deduplicate entries across multiple Makefiles.

Added QEMU and linux/amd64,linux/arm64 platforms to the publish workflow so images work on Apple Silicon.

@sourceant

sourceant Bot commented Mar 8, 2026

Copy link
Copy Markdown

Code Review Summary

This PR enhances the Laravel Makefile to be more extensible for downstream projects and adds a robust multi-arch (AMD64/ARM64) build pipeline using GitHub Actions native ARM runners.

🚀 Key Improvements

  • Added hook variables (EXTRA_PORTS_SCRIPT, EXTRA_UP_INFO, etc.) to laravel.mk.
  • Implemented multi-platform Docker builds using a merge-manifest strategy.
  • Improved make help to handle multiple included Makefiles without duplicating targets.

💡 Minor Suggestions

  • Refined the regex for target detection in make help.
  • Standardized DC variable shell execution.

@sourceant sourceant Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review complete. No specific code suggestions were generated. See the overview comment for a summary.

@nfebe nfebe force-pushed the enhance/extensible-makefile-multiarch branch from 1193b4c to 97e3de8 Compare March 8, 2026 17:37

@sourceant sourceant Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review complete. See the overview comment for a summary.

Comment thread laravel.mk Outdated
Comment thread laravel.mk Outdated
Added hook variables (EXTRA_PORTS_SCRIPT, EXTRA_UP_INFO, TEST_CMD,
LINT_CMD) so projects can customize behavior via include without
overriding targets. Fixed DC variable to strip comments from .ports
file and help target to deduplicate entries across multiple
Makefiles.

Added QEMU and linux/amd64,linux/arm64 platforms to the publish
workflow so images work on Apple Silicon.
@nfebe nfebe force-pushed the enhance/extensible-makefile-multiarch branch from 97e3de8 to 4067a6a Compare March 8, 2026 17:50

@sourceant sourceant Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review complete. No specific code suggestions were generated. See the overview comment for a summary.

@nfebe nfebe merged commit 55149cf into main Mar 8, 2026
3 checks passed
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.

1 participant