Skip to content

API: Upgrade to hamcrest/hamcrest-php v2.0.0#41

Open
madmatt wants to merge 1 commit into
hafriedlander:masterfrom
madmatt:pulls/add-composer
Open

API: Upgrade to hamcrest/hamcrest-php v2.0.0#41
madmatt wants to merge 1 commit into
hafriedlander:masterfrom
madmatt:pulls/add-composer

Conversation

@madmatt

@madmatt madmatt commented Jun 19, 2019

Copy link
Copy Markdown
  • Pull in via composer rather than directly in-lining Hamcrest (fixes composer require hamcreast  #38)
  • Fix references to old classnames (Hamcrest_Matcher -> Hamcrest\Matcher and Hamcrest_Core_IsInstanceOf -> Hamcrest\Core\IsInstanceOf)
  • Resolves the very important Pronunciation #37 (pronunciation of phockito)
  • Fixes PhockitoHamcrestTest::testCanStubByType() having the test args around the wrong way (actual, expected instead of expected, actual)

- Fix references to old classnames (Hamcrest_Matcher -> Hamcrest\Matcher and Hamcrest_Core_IsInstanceOf -> Hamcrest\Core\IsInstanceOf)
- Pull in via composer rather than directly in-lining Hamcrest (fixes hafriedlander#38)
- Resolves the very important hafriedlander#37
- Fixes PhockitoHamcrestTest::testCanStubByType() having the test args around the wrong way (actual, expected instead of expected, actual)
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.

composer require hamcreast

1 participant