Removed duplicate dependencies.

[danepowell]

The template composer.json defines a direct dependency on several packages that are already dependencies of other packages:

• Drupal Console (dependency of BLT)
• Drupal Coder (BLT)
• PHPUnit (BLT)
• PHPCS (BLT)
• Composer installers (Drupal Console and Lightning)

Because these dependencies are effectively defined twice, composer throws a warning when you run composer update on a BLT project:

Dependency "drupal/coder" is also a root requirement, but is not explicitly whitelisted. Ignoring.

Right now I think this is a harmless warning, but it could lead to unpredictable behavior if BLT, Drupal console, or Lightning updates or patches one of these dependencies.

Seems like the best solution is to remove them from the project template composer.json, and just allow upstream projects to manage them.

[danepowell]

I've confirmed that removing these dependencies from a generated BLT project doesn't actually change the composer.lock, confirming that they were basically superfluous.

[grasmash]

Perhaps we should move a few of these to BLT's composer.json:

• composer/installers
• cweagans/composer-patches

[danepowell]

Yeah, composer-installers comes from both Lightning and Drupal Console. But since I think BLT invokes it directly, we should move it to BLT's composer.json.