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

Instanceof is migration #194

Merged
merged 11 commits into from Jul 12, 2019
Merged

Conversation

fredemmott
Copy link
Contributor

No description provided.

Copy link
Collaborator

@lexidor lexidor left a comment

Choose a reason for hiding this comment

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

This is as feature complete as I would expect this migration to be. It works with generics (both built-in) and userland. It may need you to add hh_fixme's in cases where the wildcard does not infer the type of one of the generics. This is out of scope for this migration.

@fredemmott
Copy link
Contributor Author

There's --add-fixmes ; that's probably appropriate for some projects but others will want manual inspection

@fredemmott fredemmott merged commit 1e8aea2 into hhvm:master Jul 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants