Skip to content
This repository has been archived by the owner on May 25, 2020. It is now read-only.

Upgrade to use 2.0 of symfony/psr-http-message-bridge #41

Merged
merged 3 commits into from
May 8, 2020

Conversation

gpn273
Copy link
Contributor

@gpn273 gpn273 commented May 7, 2020

Q A
Bug fix? Yes
New feature? No

Description

Laravel Passport, as of versions greater than 8.4, have now upgraded to use symfony/psr-http-message-bridge ^2.0. This now presents a conflict, and whether to decide to lock the Laravel Passport version or roadrunner-laravel. I have bumped the version from ^1.2 to ^2.0 and ran current PHPUnit tests, 63 has passed.

Checklist

  • I have performed a self-review of my own code
  • I have made changes in CHANGELOG.md file

I have updated the CHANGELOG.md to version 3.2.1 to state the upgrade to symfony/psr-http-message-bridge ^2.0

@gpn273 gpn273 requested a review from tarampampam as a code owner May 7, 2020 23:10
@gpn273 gpn273 changed the title Upgrade to use 2.0 of message bridge Upgrade to use 2.0 of symfony/psr-http-message-bridge May 7, 2020
composer.json Outdated Show resolved Hide resolved
@tarampampam tarampampam self-assigned this May 8, 2020
@codecov-io
Copy link

codecov-io commented May 8, 2020

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #41   +/-   ##
=========================================
  Coverage     97.70%   97.70%           
  Complexity      105      105           
=========================================
  Files            23       23           
  Lines           262      262           
=========================================
  Hits            256      256           
  Misses            6        6           

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 984cba6...ad2767b. Read the comment docs.

@tarampampam tarampampam merged commit 6e0aceb into avto-dev:master May 8, 2020
@tarampampam
Copy link
Contributor

Thx for your contribution! 👍

@tarampampam
Copy link
Contributor

Released: https://github.com/avto-dev/roadrunner-laravel/releases/tag/v3.2.1

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

Successfully merging this pull request may close these issues.

3 participants