composer/installers installs packages to the correct location based on the specified package type. Drupal core composer/installers since forever (#2373197: Reference Composer Installers as a dependency of a Drupal project) but it is not configured properly so far. Contrib modules should go to modules/contrib. Contrib themes should go to themes/contrib etc.
Add a good default configuration for installer-paths. This will improve the ootb experience for composer based workflows.
| Comment | File | Size | Author |
|---|---|---|---|
| #11 | 2813897-11.patch | 1.15 KB | webflo |
| #7 | 2813897-7.patch | 1.21 KB | webflo |
| #2 | 2813897-2.patch | 662 bytes | webflo |
Comments
Comment #2
webflo commentedCopy from https://github.com/drupal-composer/drupal-project/blob/8.x/composer.json without the web prefix.
Comment #3
webflo commentedComment #4
bojanz commented+1 for RTBC from me.
I've seen support requests about this in the Commerce, Address, Search API Solr issue queues. People expect modules/contrib to be used, but it isn't. For 8.2.x we'll want to have a documentation page, but for 8.3.x we can give people one less thing to configure.
Comment #5
timmillwoodWorks for me!
Comment #7
webflo commentedWith updated composer hash.
Comment #8
webflo commentedComment #9
mglaman+1 This is going to simplify the lives of contributed project developers and writing documentation. If we say something is a best practice, core should reflect that :)
Comment #10
tstoecklerCan we leave
librariesout, for now a least? That is somewhat contentious, see #667058: Add a libraries folder with a README.txt in it to DRUPAL_ROOT.Comment #11
webflo commentedAlright, this time without libraries.
Comment #12
tstoecklerThank you!
Comment #13
alexpottCommitted 2098296 and pushed to 8.3.x. Thanks!