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

Release 2.0.0 #253

Closed
szepeviktor opened this issue Nov 11, 2024 · 9 comments
Closed

Release 2.0.0 #253

szepeviktor opened this issue Nov 11, 2024 · 9 comments

Comments

@szepeviktor
Copy link
Owner

szepeviktor commented Nov 11, 2024

What does it take to release 2.0.0?
Do we have v1-only features?
What is lost in v2?

"php-stubs/wordpress-stubs": "^6.6.2",

@IanDelMar
Copy link
Contributor

v1-only is support for PHP 7.2 and below, PHPStan below 2.x, older versions of WP(stubs), and dynamic return type extensions for functions with filterable arguments that were not transferred. (I am not certain whether the incorrect ones were fixed in v1 as well; I’ll verify this over the next few days).

I think it would be best to resolve the _get_list_table() issue (php-stubs/wordpress-stubs#257) before releasing 2.0.0.

@LeTraceurSnork
Copy link

Could you please sync versions with PHPStan original package starting from v2 please? I mean, its a bit unhandy that PHPStan last v1 minor is 1.12 while this package is 1.3, it's a bit confusing

@szepeviktor
Copy link
Owner Author

Could you please sync versions with PHPStan original package

Vote time!

@IanDelMar
Copy link
Contributor

IanDelMar commented Nov 12, 2024

While I completely understand @LeTraceurSnork's point, halting progress on szepeviktor/phpstan-wordpress until PHPStan advances may not be in the best interests of the contributors and users. Any new feature, such as a return type extension, qualifies for a minor release, meaning we would need to wait for PHPStan’s next minor update. Fixes for extensions should be released as patches, but we would be unable to do so until PHPStan issues its next patch, minor, or even major release. Ultimately, semantic versioning becomes pointless for szepeviktor/phpstan-wordpress. After all, one look at the composer.json file tells you which PHPStan versions are applicable to the given version of szepeviktor/phpstan-wordpress.

@LeTraceurSnork
Copy link

What about extended semver like X.Y.Z.P, where P is suffix for your own version, and Z - is latest patch of parent package

@szepeviktor
Copy link
Owner Author

@LeTraceurSnork All I can promise you is when I see a Merge button I click on it.
It would be great not to be restricted by PHPStan's development.
Have you seen other extensions' release versions?

@IanDelMar
Copy link
Contributor

What about extended semver like X.Y.Z.P, where P is suffix for your own version, and Z - is latest patch of parent package

Unless we immediately drop support for older versions of PHPStan right after a release (don't go breaking Viktor's heart 🙏), I think this will create more confusion than it resolves. If X.Y.Z is always the latest version, others will not only have to check whether older versions are supported (which they already need to do), but it also gives the false impression that szepeviktor/phpstan-wordpress is somehow bound to X.Y.Z, which is not the case.

@LeTraceurSnork
Copy link

Ok, suggestion withdrawn 🙈

@szepeviktor
Copy link
Owner Author

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

3 participants