- 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 Ahmed Aziz ABBASSI
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
over 1 year ago 10:57am 6 April 2023 - Issue was unassigned.
- Status changed to Needs review
over 1 year ago 12:59pm 7 April 2023 - Assigned to omkar.podey
- Assigned to yash.rode
- Status changed to Needs work
over 1 year ago 10:13am 10 April 2023 - Assigned to omkar.podey
- Status changed to Needs review
over 1 year ago 10:53am 10 April 2023 - Issue was unassigned.
- Assigned to tedbow
- Assigned to yash.rode
- Status changed to Needs work
over 1 year 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
over 1 year 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
over 1 year ago Not currently mergeable. - Status changed to RTBC
over 1 year 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
over 1 year ago 712 pass - last update
over 1 year ago 793 pass