- Issue created by @tedbow
- ๐บ๐ธUnited States tedbow Ithaca, NY, USA
Should update this test now after ๐ Update FixtureManipulator to work with InstalledPackagesList, real composer show command Fixed to not confirm FixtureManipulator works by inspecting the installed.json and installed.php file? We no longer edit these file directly
We could run actually
composer show
command and confirm the packages are correct. - ๐ง๐ชBelgium wim leers Ghent ๐ง๐ช๐ช๐บ
+1 to everything you said!
But I think we should do as part of ๐ Remove our runtime dependency on composer/composer: remove ComposerUtility Fixed .
- ๐น๐ณTunisia azizos
Ahmed Aziz ABBASSI โ made their first commit to this issueโs fork.
- @ahmed-aziz-abbassi opened merge request.
- Assigned to yash.rode
- Status changed to Needs work
about 2 years ago 10:57am 6 April 2023 - Issue was unassigned.
- Status changed to Needs review
about 2 years ago 12:59pm 7 April 2023 - Assigned to omkar.podey
- Assigned to yash.rode
- Status changed to Needs work
about 2 years ago 10:13am 10 April 2023 - Assigned to omkar.podey
- Status changed to Needs review
about 2 years ago 10:53am 10 April 2023 - Issue was unassigned.
- Assigned to tedbow
- Assigned to yash.rode
- Status changed to Needs work
about 2 years ago 2:36pm 10 April 2023 - ๐บ๐ธUnited States tedbow Ithaca, NY, USA
I fixed a few things and then made few more comments
- Issue was unassigned.
- Status changed to Needs review
about 2 years ago 7:02am 11 April 2023 - Assigned to tedbow
- Assigned to phenaproxima
- ๐บ๐ธUnited States tedbow Ithaca, NY, USA
@yash.rode looks good.
I assigning to @phenaproxima for final review since also work on this
- Issue was unassigned.
- ๐บ๐ธUnited States tedbow Ithaca, NY, USA
@Wim Leers could also review this so I will unassign
- Open on Drupal.org โCore: 10.1.x + Environment: PHP 8.1 & MySQL 8last update
about 2 years ago Not currently mergeable. - Status changed to RTBC
about 2 years ago 8:49am 17 April 2023 - ๐ง๐ชBelgium wim leers Ghent ๐ง๐ช๐ช๐บ
Conflicts with ๐ In some situations, pass JSON_THROW_ON_ERROR when decoding JSON Postponed: needs info .
Fixed merge conflict.
- last update
about 2 years ago 712 pass - last update
about 2 years ago 793 pass