? ? Success

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
28 Jun 2017

Pull Request for Issue #16881.

Summary of Changes

Composer has an optimized mode, where they recommend to use in production. More information can be found in the composer documentation.

This pr activates that mode, which leads to bigger class map files and less IO. More infromation can be found in issue #16881.

Testing Instructions

Just log in on the back end and open some articles on the front end.

Expected result

All should work as expected, just faster.

Actual result

All should work as expected.

avatar laoneo laoneo - open - 28 Jun 2017
avatar laoneo laoneo - change - 28 Jun 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Jun 2017
Category External Library Libraries
avatar joomla-cms-bot joomla-cms-bot - change - 26 Jul 2017
Category External Library Libraries External Library Composer Change Libraries
avatar laoneo
laoneo - comment - 26 Jul 2017

As with the merge of #16754, this one can safely be merged, as a composer install in optimized mode was run there. This pr only adds the required configuration option to the composer.json file that the next composer update will not overwrite the optimized classmap files.
I did another composer install and there are some test classes showing up in the diff which should not being mapped.

avatar mbabker
mbabker - comment - 28 Jul 2017

I'll merge this after #16780 and #17081 since theoretically this should be the simplest of the PRs to deconflict.

avatar laoneo
laoneo - comment - 28 Jul 2017

I can run then another composer install, no worries.

avatar laoneo
laoneo - comment - 10 Aug 2017

Merged with staging too.

avatar mbabker mbabker - change - 10 Aug 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-08-10 23:36:05
Closed_By mbabker
avatar mbabker mbabker - close - 10 Aug 2017
avatar mbabker mbabker - merge - 10 Aug 2017

Add a Comment

Login with GitHub to post a comment