Skip to content

Commit

Permalink
simplified box v3 version - be sure to have all dependencies elements
Browse files Browse the repository at this point in the history
  • Loading branch information
llaville committed Nov 26, 2018
1 parent 3ea63b9 commit 086fee1
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 55 deletions.
55 changes: 0 additions & 55 deletions box.json

This file was deleted.

12 changes: 12 additions & 0 deletions box.json.dist
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
{
"directories": ["src", "vendor"],

This comment has been minimized.

Copy link
@theofidry

theofidry Nov 26, 2018

is there a real need to manage those manually instead of letting box auto-discovering them?

This comment has been minimized.

Copy link
@llaville

llaville Nov 26, 2018

Author Owner

I've found some strange issues, that did not include all dependencies. Don't know yet why. This is the reason of such config.
Even if I don't like it, because the total size increased x2, I'm sure to have all dependencies, and got a product running as expected.

This comment has been minimized.

Copy link
@theofidry

theofidry via email Nov 26, 2018

"compression": "GZ",
"dump-autoload": false,
"force-autodiscovery": true,
"files": [
"manifest.txt",
"LICENSE",

This comment has been minimized.

Copy link
@theofidry

theofidry Nov 26, 2018

you could actually have this file and the previous one as files-bin

"vendor/autoload.php"

This comment has been minimized.

Copy link
@theofidry

theofidry Nov 26, 2018

hm this is weird, it shouldn't be needed

],
"stub": "phar-stub.php"
}

0 comments on commit 086fee1

Please sign in to comment.