GitExcluder should not ignore .git directories that belong to packages installed by Composer

Manually did phpcs locally, all green 👍

The follow-up I requested in #56 was created: 📌 Add new InstalledPackagesList which does not rely on Composer API to get package info Fixed .

Needs work

See question I put on the MR about the need for the new fixture

We just put a lot of effort into getting rid of most fixture folders so I want make sure we need this one.

I agree, we can definitely create addDotGitFolder

I like it! 😄

Postponing on 📌 Add addDotGitFolder functionality to \Drupal\fixture_manipulator\FixtureManipulator Fixed .

This blocks 📌 Ensure that intentionally failed test stage directories are deleted after tests complete Fixed , so tagging.

If the test above passes it proves the compatibility with 📌 Add addDotGitFolder functionality to \Drupal\fixture_manipulator\FixtureManipulator Fixed

fixed 📌 Add addDotGitFolder functionality to \Drupal\fixture_manipulator\FixtureManipulator Fixed

I'll get this across the finish line while @omkar.podey is on PTO.

First, git merge origin/8.x-2.x…

Since #71:

1. removed the fixture that was possible to remove thanks to 📌 Add addDotGitFolder functionality to \Drupal\fixture_manipulator\FixtureManipulator Fixed
2. and then reviewed + minimized the diff:
   1. fixed an incorrectly resolved merge conflict (by deleting code 😁)
   2. reverted the LockFileValidator ::stopPropagation() additions which seem to be obsolete now.
   3. reverted the StagedDBUpdateValidatorTest change because that seems to be obsolete too now

That took this MR down from 13 changed files to a mere 3!

phpcs passes locally BTW 👍

Thanks everyone! A lot of work but a big win we don't message with git folder managed by Composer!

Automatically closed - issue fixed for 2 weeks with no activity.