Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Composer version 2 #27

Merged
merged 2 commits into from
Apr 16, 2020
Merged

Add support for Composer version 2 #27

merged 2 commits into from
Apr 16, 2020

Conversation

Ayesh
Copy link
Contributor

@Ayesh Ayesh commented Apr 15, 2020

Hi John,

Composer version 2 has composer-plugin-api version 2. This PR updates the version constraint to ^1 || ^2 so we can support both composer versions.

See What's new in Composer 2 and UPGRADE-2.0 for more changes in API. Empty methods \johnpbloch\Composer\WordPressCorePlugin::deactivate() and \johnpbloch\Composer\WordPressCorePlugin::uninstall() are added to make it compatible both versions.

Related: composer/composer#8726

Thank you.

Composer version 2 has `composer-plugin-api` version 2. This PR updates the version constraint to `^1 || ^2` so we can support both composer versions.

See [What's new in Composer 2](https://php.watch/articles/composer-2) and [UPGRADE-2.0](https://github.com/composer/composer/blob/master/UPGRADE-2.0.md#for-integrators-and-plugin-authors) for more changes in API. Empty methods `\johnpbloch\Composer\WordPressCorePlugin::deactivate()` and `\johnpbloch\Composer\WordPressCorePlugin::uninstall()` are added to make it compatible both versions.
@codecov-io
Copy link

codecov-io commented Apr 15, 2020

Codecov Report

Merging #27 into master will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #27   +/-   ##
=========================================
  Coverage     94.59%   94.59%           
  Complexity       19       19           
=========================================
  Files             2        2           
  Lines            37       37           
=========================================
  Hits             35       35           
  Misses            2        2           
Impacted Files Coverage Δ Complexity Δ
src/johnpbloch/Composer/WordPressCorePlugin.php 60.00% <0.00%> (ø) 3.00 <2.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bb894f3...7410911. Read the comment docs.

@johnpbloch
Copy link
Owner

Howdy @Ayesh,

Thanks so much for taking the time to send this PR in! I ended up causing a few conflicts with your work in the process of getting the test suite to work with the latest version of PHPUnit, so I'm not sure how it's going to end up attributing the line changes to you when I resolve them and merge this in, but just in case I also gave you text credit in the changelog at the bottom of the readme.

@johnpbloch johnpbloch merged commit 237faae into johnpbloch:master Apr 16, 2020
@Ayesh
Copy link
Contributor Author

Ayesh commented Apr 17, 2020

Awesome, thank you so much 🙏.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants