Skip to content
This repository has been archived by the owner on Oct 1, 2021. It is now read-only.

feat: add PHAR generation on tag #30

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

owenvoke
Copy link
Member

@owenvoke owenvoke commented Aug 14, 2020

This adds a Box PHAR workflow to generate a PHAR for Drift when a new tag is pushed.

Check out https://github.com/owenvoke/drift/actions/runs/208504306 for an example.

What this does:

  • Creates PHAR
  • Adds it as an "artifact" which lets you download the generated PHAR through GitHub's UI
  • Creates a release on GitHub and adds the PHAR as an asset

Issues

  • Probably need to get PHAR scoping working before merging, I had issues locally (see CI workflow here).
    This should now be working...
  • The PHAR does not actually work (🤦🏻) as the RectorRunner executes the Rector binary, which is inside the PHAR so can't be run...

Closes #21

@owenvoke owenvoke marked this pull request as draft August 14, 2020 12:05
@olivernybroe
Copy link
Member

Awesome, hope you solve the issues so the phar works 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a Phar with dependencies
2 participants