Skip to content
This repository has been archived by the owner on Sep 13, 2022. It is now read-only.

chore: added php 7.3-7.4 compatibility checker #132

Merged
merged 1 commit into from
Feb 18, 2022
Merged

Conversation

arzola
Copy link
Contributor

@arzola arzola commented Feb 16, 2022

This PR adds an extra check to ensure our codebase doesn't have any deprecated or non compatible function within our supported PHP versions

https://github.com/pressbooks/private/issues/815

@private-packagist
Copy link

composer.lock

Package changes

Package Operation From To Changes
phpcompatibility/php-compatibility add - 9.3.5 view code
squizlabs/php_codesniffer add - 3.5.8 view code

Dev Package changes

Package Operation From To Changes
phpcompatibility/php-compatibility remove 9.3.5 - -
squizlabs/php_codesniffer remove 3.5.8 - -

Settings · Docs · Powered by Private Packagist

@codecov
Copy link

codecov bot commented Feb 16, 2022

Codecov Report

Merging #132 (8d887fd) into dev (c268f0f) will not change coverage.
The diff coverage is n/a.

❗ Current head 8d887fd differs from pull request most recent head 53811b2. Consider uploading reports for the commit 53811b2 to get more accurate results

Impacted file tree graph

@@            Coverage Diff            @@
##                dev     #132   +/-   ##
=========================================
  Coverage     79.09%   79.09%           
  Complexity      327      327           
=========================================
  Files             9        9           
  Lines          1172     1172           
=========================================
  Hits            927      927           
  Misses          245      245           

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 c268f0f...53811b2. Read the comment docs.

@SteelWagstaff SteelWagstaff merged commit 54a7793 into dev Feb 18, 2022
@SteelWagstaff SteelWagstaff deleted the php74-check branch February 18, 2022 15:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants