loadsys / loadsys/CakePHP-ConfigReadShell
Probably don't need `composer/installers` under composer v2?
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 3
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
Seems like this might be the cause of `composer install` creating a `/Plugin` root folder and installing every package that [has a `type: cakephp-plugin` in their own `composer.json`](https://github.com/loadsys/CakePHP-ConfigReadShell/blob/7a4a69d0f65c3ba70f8eccb0c768becacf32dd6a/composer.json#L4) into that (tracked) folder.
It's not clear to me how to detect whether this plugin is being `composer install`ed via composer v1 or v2, but on the `master / 4.x` branch it's probably safe to [remove `composer/installers` from the `composer.json` file](https://github.com/loadsys/CakePHP-ConfigReadShell/blob/7a4a69d0f65c3ba70f8eccb0c768becacf32dd6a/composer.json#L25) in THIS project.
The issue is fixed when a different project that runs `composer require loadsys/cakephp-config-read:~4.0` should NOT end up with a root `Plugin/` folder when running `composer install`.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with composer.json, especially the composer/installers entry around the referenced line, and reproduce installation from another project requiring loadsys/cakephp-config-read:~4.0. Confirm that the dependency is not needed for the master/4.x setup, then verify that composer install no longer creates a root Plugin/ folder or installs cakephp-plugin packages there.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- build-system
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100