Skip to content

Fix rewinddir() return type#192

Merged
ondrejmirtes merged 1 commit intophpstan:masterfrom
steevanb:issue-3232
May 4, 2020
Merged

Fix rewinddir() return type#192
ondrejmirtes merged 1 commit intophpstan:masterfrom
steevanb:issue-3232

Conversation

@steevanb
Copy link
Copy Markdown
Contributor

@steevanb steevanb commented May 4, 2020

@ondrejmirtes
Copy link
Copy Markdown
Member

Thank you!

@phil-davis
Copy link
Copy Markdown
Contributor

Note: these sort of changes are not "strictly backwards-compatible".
owncloud/core#37335 - CI broke because it was "expecting this error and ignoring it" but now the bug has been fixed and so the error does not happen and phpstan complains.

I don't know what you should do about these sort of fixes - actually the fix is "a good thing" (tm) - but suddenly people's CI breaks.

@ondrejmirtes
Copy link
Copy Markdown
Member

@phil-davis Yeah, my stance is that most people are glad that PHPStan finds new bugs in their code.

If you want to minimize chances your CI will break, you can:

  1. Set reportUnmatchedIgnoredErrors to false to prevent the error you're talking about.
  2. Require specific PHPStan version 0.12.22 instead of a looser constraint like ^0.12.22 if you really want your build to stay green.

PHPStan is honoring backwards compatibility in the sense that your configuration file will not break in minor versions (0.12.x - when x increases), and also that the extensions API remains the same in minor versions.

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.

3 participants