Problem/Motivation

In preparation to expose Drupal Components outside of Drupal we added a composer.json file to every component and a "replace" section to core/composer.json.

Right now drupal/core-assertion is missing from the drupal/core replace section.

Proposed resolution

- Fix the drupal/core replace section to have the drupal/core-assertion component declared.
- Add any external drupal/core-assertion requirements to drupal/core/composer.json and have composer update the other drupal/core/composer.* files.
- Add to Drupal\Tests\ComposerIntegrationTest::getPaths().

CommentFileSizeAuthor
#4 2752127.patch510 bytesjalpesh

Comments

Eric_A created an issue. See original summary.

eric_a’s picture

eric_a commented
Issue summary: View changes
jalpesh’s picture

jalpesh commented
Assigned: Unassigned » jalpesh
jalpesh’s picture

jalpesh commented
Status: Active » Needs review
StatusFileSize
new 2752127.patch510 bytes

Here we go with patch. Thanks

eric_a’s picture

eric_a commented
Status: Needs review » Reviewed & tested by the community
Related issues: +#2600716: Add composer.json to \Drupal\Component\Assertion component., +#2337283: Add a composer.json file to every component

Thanks!

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 4: 2752127.patch, failed testing.

eric_a’s picture

eric_a commented
Status: Needs work » Reviewed & tested by the community
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed cc54a9d and pushed to 8.1.x. Thanks!

  • alexpott committed 9e73da7 on 8.2.x 
    Issue #2752127 by jalpesh, Eric_A: drupal/core-assertion is missing from...

  • alexpott committed cc54a9d on 8.1.x 
    Issue #2752127 by jalpesh, Eric_A: drupal/core-assertion is missing from...

Status: Fixed » Closed (fixed)

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