Skip to content

Allow the parsing of composer 2 installed.json#212

Closed
philipheimboeck wants to merge 1 commit into
EdgedesignCZ:masterfrom
webgears:composer-2-fix
Closed

Allow the parsing of composer 2 installed.json#212
philipheimboeck wants to merge 1 commit into
EdgedesignCZ:masterfrom
webgears:composer-2-fix

Conversation

@philipheimboeck

Copy link
Copy Markdown

This PR allows to parse the installed.json generated by composer 2 as the format changed.
Due to this change it was not possible anymore to get the current versions of the installed tools.

https://getcomposer.org/upgrade/UPGRADE-2.0.md

vendor/composer/installed.json format changed:

  • packages are now wrapped into a "packages" top level key instead of the whole file being the package array

@ndench

ndench commented Nov 29, 2020

Copy link
Copy Markdown
Contributor

Thanks @philipheimboeck! I'm running into this issue as well.

Do you have any idea why CI would be failing?

  [RuntimeException] 
  Could not scan for classes inside "C:\projects\phpqa\vendor/sebastian/phpcpd/src/" which does not appear to 
  be a file nor a folder   

Looks like some issue with running on windows tbh. I doubt this PR caused it though.
The error isn't present on other PRs, but they are all much older. Maybe it's an issue with composer 2.0?

@ndench

ndench commented Dec 16, 2020

Copy link
Copy Markdown
Contributor

I've been using this fork for a couple of weeks now and it works fine. Any chance we can get this merged soon?

@ptondereau

Copy link
Copy Markdown

any news on it?

@zdenekdrahos

zdenekdrahos commented Feb 2, 2021

Copy link
Copy Markdown
Member

It's going to be fixed in 0e0ce89.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants