Skip to content

Test under 8.2+#12

Merged
Firehed merged 4 commits into
masterfrom
expand-tests
Apr 5, 2023
Merged

Test under 8.2+#12
Firehed merged 4 commits into
masterfrom
expand-tests

Conversation

@Firehed

@Firehed Firehed commented Apr 5, 2023

Copy link
Copy Markdown
Owner

Expand test matrix to latest versions of PHP.

Fix a couple of incidental static analysis warnings.

@codecov

codecov Bot commented Apr 5, 2023

Copy link
Copy Markdown

Codecov Report

Patch coverage has no change and project coverage change: +0.11 🎉

Comparison is base (67d7436) 94.19% compared to head (52f9b81) 94.30%.

❗ Current head 52f9b81 differs from pull request most recent head c64ebff. Consider uploading reports for the commit c64ebff to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #12      +/-   ##
============================================
+ Coverage     94.19%   94.30%   +0.11%     
  Complexity       49       49              
============================================
  Files             4        4              
  Lines           155      158       +3     
============================================
+ Hits            146      149       +3     
  Misses            9        9              

see 2 files with indirect coverage changes

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

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@Firehed Firehed changed the title Test under 8.2 Test under 8.2+ Apr 5, 2023
@Firehed Firehed merged commit 8e8536d into master Apr 5, 2023
@Firehed Firehed deleted the expand-tests branch April 5, 2023 22:28
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