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

Breaking changes for v3 #1040

Closed
4 tasks done
muglug opened this issue Oct 27, 2018 · 5 comments
Closed
4 tasks done

Breaking changes for v3 #1040

muglug opened this issue Oct 27, 2018 · 5 comments

Comments

@muglug
Copy link
Collaborator

muglug commented Oct 27, 2018

  • Rename *Checker to *Analyzer
  • Config: change allowStringToStandInForClass default to false
  • Config: remove allowCoercionFromStringToClassConst
  • Plugin refactor cc @weirdan
@weirdan
Copy link
Collaborator

weirdan commented Oct 31, 2018

@muglug #855 as it stands does not have any breaking changes. PluginApi namespace which it introduces is an extension, old plugins would still continue to work with LegacyPlugin wrapper.

Remaining todo items are not strictly required, the PR can be merged as is . PluginApi interfaces can (and should, as currently you can only add stub files through them) be extended during v3 lifecycle.

There's couple of finishing touches I want to do before it's merged though: namespacing the extra key in package.json and renaming the PluginFacade.

@muglug
Copy link
Collaborator Author

muglug commented Nov 9, 2018

Removed CamelCase vars and properties? bc actually I think snake_case is more legible, and it's backed up by some science.

@weirdan
Copy link
Collaborator

weirdan commented Nov 9, 2018

bc actually I think snake_case is more legible, and it's backed up by some science.

What about snake_casing the inferredType then? Also, can coding standards be documented somewhere, as it deviates from commonly used style in PHP ecosystem (camelCasing everything)?

Sadly, neither php-cs-fixer nor codesniffer seem to have an option for snake_case vars/props.

@TysonAndre
Copy link
Contributor

Sadly, neither php-cs-fixer nor codesniffer seem to have an option for snake_case vars/props.

I have a custom sniffer based on another sniffer which will warn about variables/properties that seem to be camelCase, to enforce this consistency. See https://github.com/phan/phan/blob/master/internal/PHP_CodeSniffer/Sniffs/ValidUnderscoreVariableNameSniff.php and https://github.com/phan/phan/blob/9a1e89cd615ca44ffa4ccb5db69674bd9213217d/phpcs.xml#L56-L59

@ktomk
Copy link
Contributor

ktomk commented Aug 11, 2020

  • 7fd4cab - Changelog (mention: allowCoercionFromStringToClassConst)
  • dc834fd - allow_coercion_from_string_to_class_const removal / upgrade away from it

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