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: various tweaks #58

Merged
merged 2 commits into from Sep 9, 2018
Merged

Travis: various tweaks #58

merged 2 commits into from Sep 9, 2018

Commits on Jul 12, 2018

  1. Execute phpcs and security-checker from vendor/bin

    Geoff Appleby committed Jul 12, 2018
    Configuration menu
    Copy the full SHA
    4f24d36 View commit details
    Browse the repository at this point in the history

Commits on Sep 7, 2018

  1. 🔧 Travis: various tweaks/fix builds

    ⚠️ PR 52 is a pre-requisite for this PR ⚠️
    
    This PR builds onto the changes made in PR 52 by gapple and PR 52 should be merged first (if accepted). After that, this PR should be rebased to make it mergeable. Please feel free to ping me to rebase.
    
    ## Proposed Changes
    
    * Fix the PHP 5.3 build by instructing Travis to use the `precise` image instead of the `trusty` image for which PHP 5.3 is no longer available.
    * Don't allow the PHP 5.3/5.4/5.5 builds to fail. After PR 52, they shouldn't be failing anymore, so there's no need to allow them to fail.
    * Run the PHPCS code style and cross-version compatibility only once.
        All PHPCS sniffs are unit tested and written to give the same results on all supported PHP versions, so there is no need for the PHPCS check to be run on each and every build.
        Using the `SNIFF` environment variable, the PHPCS check is now run only on the PHP 7.2 build.
    * Remove the command-line parameters from the PHPCS command.
        All relevant settings are already contained in the `phpcs.xml.dist` file and PHPCS will automatically use that file if found, so there is no need to pass these parameters on the command-line.
       N.B.: Previously, i.e. pre-PR 52, the PHPCS check would have been run before the `composer install` which meant that the `PHPCompatibility` standard used would not (yet) be available.
        As PR 52, changed the order in which these commands are run, the `PHPCompatibility` standard should now be available at the time of running PHPCS.
    jrfnl committed Sep 7, 2018
    Configuration menu
    Copy the full SHA
    5d84fc8 View commit details
    Browse the repository at this point in the history