Skip to content
This repository has been archived by the owner on Mar 29, 2024. It is now read-only.

Add PhpStorm advanced metadata #51

Closed
pinepain opened this issue Sep 4, 2017 · 1 comment
Closed

Add PhpStorm advanced metadata #51

pinepain opened this issue Sep 4, 2017 · 1 comment
Assignees
Milestone

Comments

@pinepain
Copy link
Member

pinepain commented Sep 4, 2017

https://confluence.jetbrains.com/display/PhpStorm/PhpStorm+Advanced+Metadata

Some methods , e.g. Script::run() may return any of Value types and it comes in handy when IDE could typehint them. There are other methods which offers rich return type variation. As I personally use PhpStorm and it's pretty decent IDE, I'd like to give it first-class support and make more developers happy.

@pinepain pinepain added this to the 0.2.0 milestone Sep 4, 2017
@pinepain pinepain self-assigned this Sep 16, 2017
@pinepain
Copy link
Member Author

pinepain commented Sep 16, 2017

It looks like PhpStorm advanced metadata is not designed to solve this problem yet.

Anyway, the user is responsible to check what value type/king he's receiving and do not depend on suggestions to keep pita low. While I'd really like to add this feature, it's not feasible at the moment and there is no clear roadmap how to achieve that. Bloating @return type with all possible subtypes is not the best idea.

Closing for now without resolution.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant