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

Travis: add "quicktest" stage for non-PR/merge builds #834

Merged
merged 1 commit into from
Aug 3, 2019

Conversation

jrfnl
Copy link
Member

@jrfnl jrfnl commented Jun 27, 2019

The "quicktest" stage will only run a CS check, linting and the unit tests against low/high PHP/PHPCS combinations. This should catch most issues.

The more comprehensive complete build against a larger combination of PHP/PHPCS combination, including code coverage will now only be run on PRs and merges to master.

To see the difference, compare the output of the push and pr Travis runs for this PR.

This should solve the Coveralls issue as identified in #829 (comment)

Note: this PR can wait until after the release and has not been included in the changelog. (release has been done)

@jrfnl jrfnl requested a review from wimg June 27, 2019 10:54
@jrfnl jrfnl added this to the 9.x Next milestone Jun 27, 2019
@jrfnl jrfnl force-pushed the feature/travis-add-quick-build-stage branch from cffc017 to ebcdcab Compare June 27, 2019 12:35
The "quicktest" stage will only run a CS check, linting and the unit tests against low/high PHP/PHPCS combinations. This should catch most issues.

The more comprehensive complete build against a larger combination of PHP/PHPCS combination, including code coverage will now only be run on PRs and merges to `master`.
@jrfnl jrfnl force-pushed the feature/travis-add-quick-build-stage branch from ebcdcab to d72e544 Compare June 27, 2019 12:36
@wimg wimg merged commit 6c784db into master Aug 3, 2019
@delete-merged-branch delete-merged-branch bot deleted the feature/travis-add-quick-build-stage branch August 3, 2019 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants