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

feat: add php 8.1 #106

Merged
merged 15 commits into from
Oct 31, 2022
Merged

feat: add php 8.1 #106

merged 15 commits into from
Oct 31, 2022

Conversation

rancoud
Copy link
Owner

@rancoud rancoud commented Oct 31, 2022

Description

Add PHP 8.1 in tests

Changelist

  • update github action test.yml
  • composer update
  • remove tests with @runInSeparateProcess, it causes error in serialize() with phpunit (only in PHP 8.1)
  • add a reset for each test in SessionTest to replace the @runInSeparateProcess

@codecov
Copy link

codecov bot commented Oct 31, 2022

Codecov Report

Merging #106 (7bba381) into master (1eef97f) will not change coverage.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##              master      #106   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity       183       191    +8     
===========================================
  Files             11        11           
  Lines            468       471    +3     
===========================================
+ Hits             468       471    +3     
Impacted Files Coverage Δ
src/Database.php 100.00% <100.00%> (ø)
src/File.php 100.00% <100.00%> (ø)
src/Redis.php 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@rancoud rancoud merged commit aab64f4 into master Oct 31, 2022
@rancoud rancoud deleted the feat-add-php81 branch October 31, 2022 17:41
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.

1 participant