Skip to content

Conversation

@iluuu1994
Copy link
Contributor

The file already exists here: https://github.com/phpstan/phpstan-src/blob/master/tests/PHPStan/Generics/data/bug2574.php

This will clash and emit a warning on case insensitive file systems.

The file already exists here: https://github.com/phpstan/phpstan-src/blob/master/tests/PHPStan/Generics/data/bug2574.php

This will clash and emit a warning on case insensitive file systems.
@ondrejmirtes
Copy link
Member

This was already tried before: https://github.com/phpstan/phpstan-src/pull/84/files

But it breaks the Windows build: https://travis-ci.org/phpstan/phpstan-src/jobs/641054293#L233

I think we have to remove the file including the test, and then add it again in another PR.

@iluuu1994
Copy link
Contributor Author

You mean both files?

@ondrejmirtes
Copy link
Member

Yeah, probably :)

@cs278
Copy link
Contributor

cs278 commented Jan 29, 2020

Easiest fix is probably just to rename the remaining file to bug-2574.php which will then resolve the conflict.

@ondrejmirtes ondrejmirtes force-pushed the master branch 2 times, most recently from 0786c63 to 9395d76 Compare February 27, 2020 02:48
@ondrejmirtes ondrejmirtes force-pushed the master branch 5 times, most recently from dc2e229 to 1d0d3ad Compare March 17, 2020 14:14
@adaamz
Copy link
Contributor

adaamz commented Apr 30, 2020

Or maybe do it in two commits?

@ondrejmirtes
Copy link
Member

Thank you!

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.

4 participants