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

Adjust PHP dependencies for composer 2.0 #273

Merged
merged 1 commit into from
Nov 3, 2020

Conversation

phil-davis
Copy link
Contributor

@phil-davis phil-davis commented Nov 3, 2020

Issue owncloud/core#38067

See issue for details - behat acceptance tests need to use PHP 7.4 for the dependencies to sort themselves out with composer 2.0

And bump PHP to 7.1 in composer.json and core min version to 10.3 - we are not going to realistically support PHP 7.0 and core 10.2 for the next release of this app.

@codecov
Copy link

codecov bot commented Nov 3, 2020

Codecov Report

Merging #273 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #273   +/-   ##
=========================================
  Coverage     68.49%   68.49%           
  Complexity      232      232           
=========================================
  Files            35       35           
  Lines           914      914           
=========================================
  Hits            626      626           
  Misses          288      288           

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 5fee460...cd72ba7. Read the comment docs.

@phil-davis phil-davis merged commit 3dca8fa into master Nov 3, 2020
@delete-merged-branch delete-merged-branch bot deleted the php-dependencies-for-composer-2.0 branch November 3, 2020 04:46
@pmaier1 pmaier1 mentioned this pull request Aug 11, 2021
41 tasks
@jnweiger jnweiger mentioned this pull request Sep 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants