Skip to content
This repository has been archived by the owner on Jul 31, 2023. It is now read-only.

Test enhancement #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

peter279k
Copy link

@peter279k peter279k commented Jan 25, 2019

Changed log

  • Add php-7.3 test in Circleci build because the php-7.3 version is stable release now.
  • Add the different released PHPUnit versions to support multiple PHP versions.
  • Add the autoload-dev block in composer.json and define the class namespaces in tests to load these classes automatically.
  • Remove useless require_once code because the classes will be loaded automatically via autoload.php.
  • The successful Circle CI build result is available here.
  • Add the suggest block in composer.json to recommend the gmp and bcmath extensions.
    It's related to issue Add a word about php gmp requirement #11.

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

@peter279k
Copy link
Author

I signed it!

@googlebot
Copy link

CLAs look good, thanks!

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.

None yet

2 participants