Skip to content
This repository was archived by the owner on Mar 13, 2024. It is now read-only.

Resolve Travis CI build issues && add PHP 7.4, 8.0#167

Open
jaymcp wants to merge 10 commits intomasterfrom
chore/update-travis
Open

Resolve Travis CI build issues && add PHP 7.4, 8.0#167
jaymcp wants to merge 10 commits intomasterfrom
chore/update-travis

Conversation

@jaymcp
Copy link
Contributor

@jaymcp jaymcp commented Jul 7, 2022

Proposed Changes

  • Add PHP 7.4 && 8.0 to Travis CI build matrix
  • Downgrade Composer to v1 in Travis CI
  • Bump Travis CI node version to 12
  • Bump Travis CI PHPCS ruleset versions
  • Resolve PHPCS errors thrown by updating PHPCS ruleset versions

@jaymcp jaymcp requested a review from ampersarnie July 7, 2022 08:56
Copy link
Member

@ampersarnie ampersarnie left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@ampersarnie ampersarnie left a comment

Choose a reason for hiding this comment

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

(This popped up in my notifications to review again, and seem to have gotten ahead of myself.)

@jaymcp
Copy link
Contributor Author

jaymcp commented Jul 15, 2022

yeah sorry mate. appreciate it

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