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

Bump to laravel/socialite:^5.2 and phpunit/phpunit:^9.0 #222

Merged
merged 3 commits into from
Aug 31, 2023

Conversation

lucasmichot
Copy link
Member

This PR bumps some dependencies

laravel/socialite:^5.2
as $usesPKCE is already used in a few providers and depends on this version

phpunit/phpunit:^9.0
as ^6.0 does not support PHP<8

@codecov
Copy link

codecov bot commented Aug 31, 2023

Codecov Report

Merging #222 (1bed9a3) into master (df5e45b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #222   +/-   ##
=========================================
  Coverage     53.49%   53.49%           
  Complexity       78       78           
=========================================
  Files            10       10           
  Lines           243      243           
=========================================
  Hits            130      130           
  Misses          113      113           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

composer.json Outdated
"support": {
"issues": "https://github.com/socialiteproviders/manager/issues",
"source": "https://github.com/socialiteproviders/manager"
},
"require": {
"php": "^8.0",
"illuminate/support": "^6.0 || ^7.0 || ^8.0 || ^9.0 || ^10.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May as well drop versions that dont support php 8 at least?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea, see 1bed9a3

@atymic atymic merged commit 8d538a6 into master Aug 31, 2023
6 checks passed
@atymic atymic deleted the feat/versions-bumps branch August 31, 2023 22:54
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.

2 participants