Port changes from pie-extensions fork - #1
Merged
Conversation
added build path option
Replace existing assets
Update dist
feat: add parseArgs function and update build process for better flag…
Optimize build process by enabling parallel execution in make commands
Smoke testing
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.
Summary
Brings across the work accumulated on
pie-extensions/pie-ext-binary-builderto this fork. The old fork is being retired in favour ofextport(re-fork rather than transfer), so the 22-commit delta frompie-extensions/mainis being applied here as a single merge.Features ported
docker-imageinput +execInDockerhelper — build and PHP detection can run inside a container viadocker runapk-packagesinput — install extra Alpine packages when building with a Docker imageparseArgshelper — splitsconfigure-flagssafely, honouring quoted argsmakebuilds.github/workflows/end-to-end.yml).github/workflows/release.yml— buildsdist/on release publish and force-updates the major-version tag (e.g.v1)dist/removed from git and added to.gitignore(now built by the release workflow)node20→node24Conflict resolution
extport/mainalready had its own merge of thebuild-path-optionPR.pie-extensions/mainhas the equivalent change via its own PR. Merge was performed with-X theirsso pie-extensions' version wins where they overlap, matching the intended end state. Thedist/index.jsanddist/index.js.mapmodify/delete conflicts were resolved by deleting (pie-extensions removes the committeddist/).Test plan
npm installcleannpm test— 43/43 passing locallyv1.0.0) to exercise the newrelease.ymland confirm it commitsdist/to the tag and force-updatesv1🤖 Generated with Claude Code