Skip to content

support for building a .phar command line utility from project sources #52

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.

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

Merged
merged 2 commits into from
Jun 7, 2016

Conversation

1ma
Copy link
Contributor

@1ma 1ma commented Jun 2, 2016

Utility tools similar to codeclimate/php-test-reporter can usually be downloaded and used as a standalone .phar package. These binaries can be useful because importing the tool into your project's composer.json might unnecessarily restrict which dependencies you can use (see issue #31), plus there's the side benefit of taking pressure off Composer's package resolving process when you do a composer update.

With that in mind I've ported the satooshi/php-coveralls box.json file into the project. These are the steps to produce one such phar:

$ cd php-test-reporter
$ curl -LSs https://box-project.github.io/box2/installer.php | php
$ composer install --no-dev
$ ./box.phar build

And here is an example of how it might be used in a .travis.yml file.

If you were to accept this PR maybe you could distribute from now on a .phar file in the Releases page, just as Satooshi does with php-coveralls.

So, there's that. Have a nice day :)

@dblandin
Copy link
Contributor

dblandin commented Jun 7, 2016

Hey @1ma,

Thank you for the PR! I like this approach and agree that phar archives would be a useful asset.

Here's what came out of the build process locally: codeclimate-test-reporter.phar.zip.

Would you mind updating the build product to codeclimate-test-reporter.phar? Then I'd be happy to merge this PR.

Thanks!

@1ma
Copy link
Contributor Author

1ma commented Jun 7, 2016

Sure, there you go.

@dblandin dblandin merged commit 9217abf into codeclimate:master Jun 7, 2016
@dblandin
Copy link
Contributor

dblandin commented Jun 7, 2016

@1ma Thanks! 👍

@1ma 1ma deleted the boxify branch June 7, 2016 17:57
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.

2 participants