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

@since comments removed, some @param missing #247

Open
kkmuffme opened this issue Oct 10, 2024 · 6 comments
Open

@since comments removed, some @param missing #247

kkmuffme opened this issue Oct 10, 2024 · 6 comments

Comments

@kkmuffme
Copy link

kkmuffme commented Oct 10, 2024

#222 and https://github.com/php-stubs/wordpress-stubs/pull/243/files

We used a custom phpstan plugin that used the @since info, but that has been removed with that PR.

In general, I would strongly advice against cosmetic changes to visitors/generators since
a) the size of the stubs file is completely irrelevant for phpstan/psalm, since the stubs we use internally in psalm are much bigger in size and it makes literally no difference in performance
b) it just causes issues for some users and does not provide a real benefit, does it?

e.g. https://github.com/php-stubs/wordpress-stubs/pull/243/files#diff-4dc94858ed12d457b90cbe75a4151de2b69d60a4693fc574a0bf8d4b71923f31L72164 this shouldn't be removed, should it?

@szepeviktor
Copy link
Member

@kkmuffme Thank you!!
I like your since tag usage.

@IanDelMar Please confirm that we have a problem.

@szepeviktor
Copy link
Member

szepeviktor commented Oct 10, 2024

this shouldn't be removed, should it?

@johnbillion This is a core bug. No docblock for get_transient_key: /* Can you fix it?

@johnbillion
Copy link
Contributor

@szepeviktor
Copy link
Member

The fastest guy in town!

@IanDelMar
Copy link
Contributor

IanDelMar commented Oct 10, 2024

https://core.trac.wordpress.org/changeset/59203

This should be resolved then. Valid docBlocks have not been removed. But does that mean that PHPStan uses information from non-docBlocks?

@kkmuffme
Copy link
Author

But does that mean that PHPStan uses information from non-docBlocks?

It does not by default, but it can - since anybody can write her own plugin.

@php-stubs php-stubs deleted a comment Oct 25, 2024
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

5 participants
@johnbillion @szepeviktor @kkmuffme @IanDelMar and others