Skip to content
This repository has been archived by the owner on Feb 20, 2024. It is now read-only.

Use vendor path to phpunit #146

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

jongotlin
Copy link
Collaborator

No description provided.

@@ -9,7 +9,7 @@ php:

before_script: composer install --prefer-source

script: phpunit --debug --coverage-text
script: vendor/phpunit/phpunit/phpunit
Copy link
Collaborator

Choose a reason for hiding this comment

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

Isnt that one too many phpunit?

I'll see if the tests fails or not.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

vendor, package, bin. It should be correct but phpunit is not imported ¯_(ツ)_/¯

@jongotlin
Copy link
Collaborator Author

Dropped test for 5.* due to memory issue. Don't think it's worth trying to fix.

PHP Fatal error: Allowed memory size of 1610612736 bytes exhausted (tried to allocate 32 bytes) in phar:///home/travis/.phpenv/versions/5.6.32/bin/composer/vendor/composer/semver/src/Constraint/Constraint.php on line 176

@dizda
Copy link
Owner

dizda commented Nov 30, 2018

Do you guys want to takeover the ownership of the bundle? As I don't maintain it anymore.

@jongotlin
Copy link
Collaborator Author

I think @Nyholm will be at better maintainer but I can sure step up if he doesn't want to.

@dizda
Copy link
Owner

dizda commented Dec 2, 2018

@jongotlin I added you as a collaborator.

@jongotlin
Copy link
Collaborator Author

Ok, thanks @dizda

@Nyholm, would you mind review this pr?

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.

3 participants