Skip to content
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

Fix Travis Builds #40

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

joemcgill
Copy link

@joemcgill joemcgill commented Nov 10, 2020

This makes several updates that fixes new WordPress warnings and brings things up to standard so that Travis builds will run.

Updates:

  • Upgrade PHPCS Composer Installer to 0.7
  • Update Travis config to enable mysql and memecached
  • Fix permissions callbacks (Add permission callbacks to REST API endpoints #39)
  • Fix PHPCS issues
  • Upgrade xwp/wp-dev-lib to 1.6.5
  • Upgrade php-coveralls to 2.4.2
  • Fix PHPUnit configuration validation

This fixes a notice thrown since WP 5.5 when a REST route is registered
without explicitly adding a permission callback.
This upgrade fixes compatibility support for Composer 2.0.
Currently Travis is using the xenial build, which doesn't auto-enablew
mysql, which causes our tests to fail. This explicitly fixes this by
enabling mysql and memcached.
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.

None yet

1 participant