Skip to content

Review code - #44

Merged
rancoud merged 2 commits into
masterfrom
feat-review-code
Apr 21, 2021
Merged

rancoud merged 2 commits into
masterfrom
feat-review-code

Conversation

@rancoud

@rancoud rancoud commented Apr 21, 2021

Copy link
Copy Markdown
Owner

Description

Review code

Changelist

  • add tests files in linter
  • encapsulate Exception into SessionException
  • add tests about SessionException
  • explain codeCoverageIgnore
  • remove token for codecov
  • remove option --no-suggest from github action
  • use multiple php versions (7.4, 8.0) for github action
  • add functions setLengthSessionID and getLengthSessionID
  • update README with session options
  • update comments on Session handler

rancoud added 2 commits April 21, 2021 23:28
* add tests files in linter
* encapsulate Exception into SessionException
* add tests about SessionException
* explain codeCoverageIgnore
* remove token for codecov
* remove option --no-suggest from github action
* use multiple php versions (7.4, 8.0) for github action
* add functions setLengthSessionID and getLengthSessionID
* update README with session options
* update comments on Session handler
@codecov

codecov Bot commented Apr 21, 2021

Copy link
Copy Markdown

Codecov Report

Merging #44 (31c73dc) into master (994aa88) will increase coverage by 0.90%.
The diff coverage is 100.00%.

Impacted file tree graph

@@              Coverage Diff              @@
##             master       #44      +/-   ##
=============================================
+ Coverage     99.09%   100.00%   +0.90%     
- Complexity      141       183      +42     
=============================================
  Files            11        11              
  Lines           442       475      +33     
=============================================
+ Hits            438       475      +37     
+ Misses            4         0       -4     
Impacted Files Coverage Δ Complexity Δ
src/ArrayManager.php 100.00% <ø> (ø) 23.00 <0.00> (ø)
src/Database.php 100.00% <100.00%> (+1.92%) 26.00 <17.00> (+10.00)
src/DatabaseEncryption.php 100.00% <100.00%> (ø) 2.00 <2.00> (ø)
src/DefaultEncryption.php 100.00% <100.00%> (ø) 2.00 <2.00> (ø)
src/DriverManager.php 100.00% <100.00%> (ø) 22.00 <7.00> (ø)
src/Encryption.php 100.00% <100.00%> (ø) 17.00 <3.00> (+7.00)
src/File.php 100.00% <100.00%> (+4.44%) 28.00 <22.00> (+6.00)
src/FileEncryption.php 100.00% <100.00%> (ø) 2.00 <2.00> (ø)
src/Redis.php 100.00% <100.00%> (+2.63%) 19.00 <12.00> (+4.00)
src/RedisEncryption.php 100.00% <100.00%> (ø) 2.00 <2.00> (ø)
... and 2 more

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 994aa88...31c73dc. Read the comment docs.

@rancoud
rancoud merged commit 687edd8 into master Apr 21, 2021
@rancoud
rancoud deleted the feat-review-code branch April 21, 2021 21:46
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