FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Comparing 2.0.0...2.1.0 · phpsu/ShellCommandBuilder · GitHub

Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: phpsu/ShellCommandBuilder
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 2.0.0
Choose a base ref
Could not load branches
Nothing to show
{{ refName }}
...
head repository: phpsu/ShellCommandBuilder
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 2.1.0
Choose a head ref
Could not load branches
Nothing to show
{{ refName }}
  • 15 commits
  • 45 files changed
  • 4 contributors

Commits on Oct 26, 2020

  1. Fix errors with assert in tests under php8

    Since PHP 7, `assert()` would throw an `AssertionError` if the ini
    option `assert.exception` was enabled, but it didn't become the default
    until PHP 8 so instead warnings were converted to an `ErrorException`.
    
    This commit updates phpunit's bootstrap script to make sure that the
    `assert.exception` option is enabled for both PHP 7 *and* 8, enabling us
    to switch the expectation to a proper `AssertionError` in the tests.
    dshoreman committed Oct 26, 2020
    Configuration menu
    Copy the full SHA
    7266ed7 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    26cc417 View commit details
    Browse the repository at this point in the history

Commits on Nov 5, 2020

  1. Update phpunit and infection php

    Bumps infection, phpunit and php-invoker to the latest versions to get
    support for php8 and the xdebug config changes.
    
    With the update to phpunit comes a new xml config schema. The new
    `coverage` section is made up of the `filter.whitelist` value (now
    'include') and the `log.coverage-*` values as children of `report`.
    dshoreman committed Nov 5, 2020
    Configuration menu
    Copy the full SHA
    0b23ed7 View commit details
    Browse the repository at this point in the history
  2. Add a test to cover mutation in CollectionTuple

    After updating the upstream image to include pcntl, Infection PHP
    flagged a mutation in `Collection\CollectionTuple::__toArray()` where
    `$this->value instanceof ShellInterface` could be changed to `false`.
    
    That results in the first half of the ternary never executing. To catch
    it, this commit adds a test for `__toArray()` that uses an instance of
    `ShellBuilder`, complimenting the existing string-based test.
    dshoreman committed Nov 5, 2020
    Configuration menu
    Copy the full SHA
    becc507 View commit details
    Browse the repository at this point in the history
  3. Update psalm

    After updating psalm, 17 errors were uncovered - all either a
    MissingConstructor error or PropertyNotSetInConstructor - largely
    depending on whether it found it in the `final` class or a parent.
    
    In all cases, the root cause was simply lack of a default value.
    dshoreman committed Nov 5, 2020
    Configuration menu
    Copy the full SHA
    cd38d35 View commit details
    Browse the repository at this point in the history
  4. Expand version constraints for 7.2 and 7.3 support

    The most recent version of Infection PHP that will run on 7.2 is 0.15.x.
    v0.19 adds support for php 8, but also drops support for versions older
    than 7.4 meaning we also need to allow for 0.18.x to get 7.3 working.
    dshoreman committed Nov 5, 2020
    Configuration menu
    Copy the full SHA
    41f1f80 View commit details
    Browse the repository at this point in the history
  5. Switch to using dual PHPUnit configs

    Phpunit 9 won't run under 7.2, but phpunit 8 doesn't understand the
    config for v9. This here is an attempt at a "dual boot" of sorts.
    
    The old config is restored to a separate file so we can pass the legacy
    config for 7.2 but still use the new config format for other versions.
    dshoreman committed Nov 5, 2020
    Configuration menu
    Copy the full SHA
    a0501aa View commit details
    Browse the repository at this point in the history
  6. Fix Infection job in travis under php7.3

    Manually installs the latest copy of phpunit 8.x for php 7.3 because
    while phpunit9 is compatible with php7.3, infection is not able to read
    the new xml config schema that phpunit uses as of v9.
    dshoreman committed Nov 5, 2020
    Configuration menu
    Copy the full SHA
    8dd0ae5 View commit details
    Browse the repository at this point in the history
  7. Fix xmllint error on phpunit.legacy.xml

    Copies the legacy config rather than moving it so that xmllint can scan
    it, which was the last thing preventing the 7.x build from passing.
    
    Since the config is always available under the same filename, we also no
    longer need to pass a custom `--configuration` flag for phpunit.
    dshoreman committed Nov 5, 2020
    Configuration menu
    Copy the full SHA
    958cfff View commit details
    Browse the repository at this point in the history
  8. Enable xdebug coverage mode for php8 in travis

    For php 8, we need to manually set the new `xdebug.mode` option to
    enable coverage mode, but it won't work if we simply do something like
    `php -d xdebug.mode=coverage composer infection` because InfectionPHP
    spawns new processes that would lose that option.
    
    To provide for this, there's the `--initial-tests-php-options` shell
    option to infection that will pass that on when it calls PHP for the
    initial test run, which is the only time we need coverage enabled.
    
    Since this is only required in PHP 8 and there are no other parts using
    the `PHPUNIT_LEGACY` env variable, we can instead test against php 7.x
    to remove the need for PHPUNIT_LEGACY entirely.
    dshoreman committed Nov 5, 2020
    Configuration menu
    Copy the full SHA
    cead4a6 View commit details
    Browse the repository at this point in the history
  9. Update composer deps

    Infection PHP hasn't released the commit adding php 8 support yet, so
    for now we reference it directly as a fallback with 1.24 preemptively
    set ready for when it's actually available.
    
    phpro/grumphp still requires ^7.3 so we can't stop ignoring platform
    requirements just yet, but enforcing 4.0.1 explicitly at least removes
    the error about 4.0.0 requiring the composer v1 plugin api package.
    dshoreman committed Nov 5, 2020
    Configuration menu
    Copy the full SHA
    2497b8f View commit details
    Browse the repository at this point in the history

Commits on Nov 6, 2020

  1. Only ignore php requirements

    Co-authored-by: Chris Ben <chris.ben9@gmail.com>
    dshoreman and ChrisB9 authored Nov 6, 2020
    Configuration menu
    Copy the full SHA
    1731971 View commit details
    Browse the repository at this point in the history
  2. Merge pull request #17 from dshoreman/fix/php8

    Add support for PHP 8 closes #17
    ChrisB9 authored Nov 6, 2020
    Configuration menu
    Copy the full SHA
    ec95a23 View commit details
    Browse the repository at this point in the history

Commits on May 23, 2025

  1. ⬆️ upgrade to php 8.4

    Kanti committed May 23, 2025
    Configuration menu
    Copy the full SHA
    23ac17c View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    0d9ae2c View commit details
    Browse the repository at this point in the history
Loading

Back | FazBrowse Home | New Git URL