Skip to content

Assume default package name if composer.json file cannot be retrieved#78

Merged
schlessera merged 4 commits intowp-cli:masterfrom
phlbnks:master
Apr 19, 2018
Merged

Assume default package name if composer.json file cannot be retrieved#78
schlessera merged 4 commits intowp-cli:masterfrom
phlbnks:master

Conversation

@phlbnks
Copy link
Copy Markdown

@phlbnks phlbnks commented Apr 13, 2018

  • Warning, not Error, when composer.json cannot be retrieved
  • Return best-guess $package_name

- Warning not Error when composer.json cannot be retreived
- Return best-guess $package_name
@phlbnks
Copy link
Copy Markdown
Author

phlbnks commented Apr 13, 2018

Realised I've broken other tests. Fixing.

Phil Banks added 2 commits April 13, 2018 14:30
@phlbnks
Copy link
Copy Markdown
Author

phlbnks commented Apr 13, 2018

Previous test failed due to OOM error, but I realised that array dereferencing isn't supported in 5.3 anyway so I've updated to remove that.

@phlbnks
Copy link
Copy Markdown
Author

phlbnks commented Apr 13, 2018

OOM again ¯_(ツ)_/¯

@schlessera
Copy link
Copy Markdown
Member

@emirpprime The OOM should now be fixed if you merge latest master.

@schlessera schlessera added bug command:package-install Related to 'package install' command labels Apr 14, 2018
@schlessera schlessera added this to the 1.0.13 milestone Apr 14, 2018
@danielbachhuber
Copy link
Copy Markdown
Member

I've merged master.

@schlessera schlessera merged commit 63bfef0 into wp-cli:master Apr 19, 2018
@schlessera
Copy link
Copy Markdown
Member

Thanks for the pull request, @emirpprime !

@schlessera schlessera changed the title Handle unavailable composer.json on private repo (issue #76) Assume default package name if composer.json file cannot be retrieved Apr 19, 2018
schlessera added a commit that referenced this pull request Jan 5, 2022
Handle unavailable composer.json on private repo

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

Labels

bug command:package-install Related to 'package install' command

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants