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

PHPUnit Version 5.7 Removed #3413

Closed
gellis opened this issue Nov 20, 2018 · 4 comments
Closed

PHPUnit Version 5.7 Removed #3413

gellis opened this issue Nov 20, 2018 · 4 comments

Comments

@gellis
Copy link

gellis commented Nov 20, 2018

Q A
PHPUnit version 5.7
PHP version 7.1
Installation Method Composer

We are using the Symfony PHPUnit Bridge and as of yesterday afternoon we are getting the following error: RuntimeException: Could not find https://github.com/sebastianbergmann/phpunit/archive/5.7.zip

It does indeed appear that version 5.7 has been removed from the archive but the Symfony PHPUnit Bridge requires that version for users on PHP 7.1.

Would it be possible to re-upload version 5.7 to the archive?

@nicolas-grekas
Copy link
Contributor

Reported also in symfony/symfony#29264 and symfony/symfony#29263
The reason is that the corresponding branches have been removed.
Please bring them back...

@sebastianbergmann
Copy link
Owner

Please do not depend on implementation details you do not control, in this case branches in PHPUnit's version control system.

@nicolas-grekas
Copy link
Contributor

nicolas-grekas commented Nov 20, 2018

The whole internet relies on things we don't control: GitHub URLs, google APIs, etc. Could you please consider reconsidering: this breaks legitimate CIs that were just fine. I suppose you won't anytime remove tags. You could though. Version branches are the same to me: they're part of history and should be kept forever.

phansys added a commit to hackzilla-project/TicketBundle that referenced this issue Dec 6, 2018
…/translation" instead of "sensio/framework-extra-bundle" (phansys)

This PR was merged into the master branch.

Discussion
----------

|Q            |A     |
|---          |---   |
|Branch       |master|
|Bug fix?     |yes   |
|New feature? |no    |
|BC breaks?   |no    |
|Deprecations?|no    |
|Tests pass?  |yes   |
|Fixed tickets|n/a   |
|License      |MIT   |
|Doc PR       |n/a   |

* Provide "translation" service via "symfony/translation" instead of "sensio/framework-extra-bundle";
* Declare "SYMFONY_PHPUNIT_VERSION" at CI tests in order to avoid issues with nonexistent PHPUnit branches (see symfony/symfony#29265, sebastianbergmann/phpunit#3413).

Commits
-------

dd8ec39 [dependencies] Provide "translation" service via "symfony/translation" instead of "sensio/framework-extra-bundle"
StijnVrolijk added a commit to forkcms/library that referenced this issue Jan 7, 2019
@rvanlaak
Copy link

FYI: phpunit v6.3 started throwing a 404

https://github.com/sebastianbergmann/phpunit/archive/6.3.zip

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

No branches or pull requests

4 participants