Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 7, 2025

Removes legacy Travis CI configuration and replaces it with a native GitHub Actions workflow for PHPUnit testing.

Changes

  • Added .github/workflows/phpunit-ci.yml: PHPUnit test matrix covering PHP 7.4-8.2 and WordPress 4.9/latest/trunk with MySQL 5.7 service container
  • Removed .travis.yml: 95-line Travis CI configuration file
  • Updated CONTRIBUTING.md: Replaced Travis CI references with GitHub Actions
  • Cleaned ignore lists: Removed .travis.yml from .gitattributes, .distignore, and deploy.sh

Workflow Features

  • PHP linting across all versions (7.4+)
  • WordPress multisite integration tests
  • Trunk builds marked as experimental (continue-on-error)
  • Dynamic plugin slug detection for portability
  • Explicit permissions (contents: read)
  • Minimum PHP version set to 7.4 to prevent PHPUnit and PHPStan errors

Existing GitHub Actions workflows for PHPCS, PHPStan, and WordPress.org deployment remain unchanged.

Original prompt

This section details on the original issue you should resolve

<issue_title>Remove travis</issue_title>
<issue_description>Remove all usage of travis and replace with Github actions. </issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Remove all usage of Travis and replace with GitHub Actions Remove Travis CI and replace with GitHub Actions Dec 7, 2025
Copilot AI requested a review from JJJ December 7, 2025 21:02
@JJJ JJJ marked this pull request as ready for review December 7, 2025 21:21
Copilot AI review requested due to automatic review settings December 7, 2025 21:21
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR successfully migrates the continuous integration setup from Travis CI to GitHub Actions, modernizing the testing infrastructure while maintaining feature parity and adding expanded PHP version coverage.

Key Changes

  • New GitHub Actions workflow with PHPUnit testing across PHP 7.2-8.2 and WordPress 4.9/latest/trunk
  • Complete Travis CI removal including configuration file and all references in ignore lists
  • Documentation updates to reflect the GitHub Actions integration

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
.github/workflows/phpunit-ci.yml New PHPUnit CI workflow with matrix testing, MySQL service container, and experimental trunk builds
.travis.yml Removed 95-line legacy Travis CI configuration
CONTRIBUTING.md Updated CI integration references from Travis-CI to GitHub Actions
.gitattributes Cleaned up export-ignore directive for .travis.yml
.distignore Removed .travis.yml from distribution ignore list
deploy.sh Removed .travis.yml from SVN ignore properties

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@JJJ
Copy link
Collaborator

JJJ commented Dec 7, 2025

@copilot set minimum PHP version to 7.4 to prevent PHPUnit and PHPStan errors.

Copy link
Contributor Author

Copilot AI commented Dec 7, 2025

@copilot set minimum PHP version to 7.4 to prevent PHPUnit and PHPStan errors.

Updated the workflow to set minimum PHP version to 7.4, removing 7.2 and 7.3 from the test matrix. Commit: 821b987

Copilot AI requested a review from JJJ December 7, 2025 21:37
@JJJ JJJ merged commit fd59530 into master Dec 7, 2025
0 of 3 checks passed
@JJJ JJJ deleted the copilot/remove-travis-and-add-github-actions branch December 7, 2025 21:55
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.

Remove travis

2 participants