-
Notifications
You must be signed in to change notification settings - Fork 2
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
Remove development dependencies from the compiled PHAR #48
Conversation
Since the Release Tool has moved from being a library to being a PHAR distributed command-line application, it makes sense to commit the composer.lock. This also allows us to depend on the composer.lock to exist when running Box (which needs the composer.lock to remove dev dependencies).
This significantly reduces the size of the compiled PHAR.
box.json
Outdated
@@ -1,13 +1,7 @@ | |||
{ | |||
"chmod": "0755", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with the last update IIRC this mode is now the default so could be removed
box.json
Outdated
"LICENSE.txt", | ||
"README.md" | ||
"directories-bin": [ | ||
"config" | ||
], | ||
"git-version": "package_version", | ||
"main": "bin/release", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
stub: true
can be removed as well since it's the default value
box.json
Outdated
"LICENSE.txt", | ||
"README.md" | ||
"directories-bin": [ | ||
"config" | ||
], | ||
"git-version": "package_version", | ||
"main": "bin/release", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since bin/release
is configured as a bin file in composer.json
, this config entry is unnecessary as well; the output one is however since the output name differs from the input one
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for reviewing! I've implemented your suggestions.
c80e4b9
to
e5da1f5
Compare
Resolves #47.
This reduces the size of the PHAR from 18.5 MB to 2.3 MB. Thanks to @theofidry for the suggestion and help.
See https://github.com/leviy/release-tool/pull/35/files/e6fe1a69beada77909a0d4816e5fec4ed8637a33#r199961790