Skip to content

Run tests using PHP 7.3 on Travis CI #1818

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

Merged
merged 2 commits into from
Jan 16, 2019
Merged

Conversation

swissspidy
Copy link
Collaborator

@swissspidy swissspidy commented Jan 16, 2019

Let's see if the tests pass with that change.

See #1813.

@googlebot googlebot added the cla: yes Signed the Google CLA label Jan 16, 2019
…3-for-travis

* 'develop' of github.com:ampproject/amp-wp:
  Correct license in composer.json as GPL-2.0-or-later
  Consolidate translator comments for forgotten URLs string
  Drop unnecessary sanitize_hex_color() polyfill and rename amp-post-template-actions.php to amp-post-template-functions.php.
  Fix composer.json indentation.
  Ensure all dependencies are PHP 5.4 compatible.
  Ensure FasterImage allows to set its user agent.
  Eliminate PHP 5.3-compatibility workarounds for closures
  Eliminate PHP 5.3-specific testsuite
  Ensure that Travis will npm install and composer install for each build
  Restore linebreaks in readme.md
  Exclude patches from build
  Improve phpdoc; catch Exception from fetch_images; add todo
  Remove unnecessary support for  argument when fetching image dimensions and deprecate it on the public method.
  Fix FunctionCallSignature for new WP code style
  Fix MultipleStatementAlignment in autoloader manifest
  Pull in FasterImage library via Composer and apply manual changes via patch file.
  Remove now unnecessary FastImage library.
  Update minimum required PHP version to 5.4.
@westonruter westonruter added this to the v1.1 milestone Jan 16, 2019
@westonruter westonruter merged commit 7c0dba8 into develop Jan 16, 2019
@westonruter westonruter deleted the 1813-php73-for-travis branch January 16, 2019 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Signed the Google CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants