php-ast
php-ast copied to clipboard
nikic
Publish package on Packagist / Composer
Hello,
I think it will be a good idea to publish this package on Packagist, for the PHP stub?
I'd like to bump this and even suggest that maybe PECL would be a good option. I 100% support the efforts of #5 but until that gets more traction this should be pursued. That's just my opinion, but I think there's a huge benefit to having easier access to this extension.
With testing environments like gitlabci, travis and etc PECL would make this a lot easier to run automated tests with tools required ext-ast. If distributing that with composer is an option as well then I would support that one too! Looking forward to any info on this or any updates.
I'd be happy to assist with getting this into Composer or PECL if that would make it easier to get this moving!
IMHO, this is not a PHP package per se and it shouldn't be published on packagist.
@drupol I think the intent of this was to have easier access - per my comment PECL made more sense though. Now that' it's on PECL this could probably be closed for either: a) being fulfilled, or b) for not being the proper route.
I believe the idea is to publish it on composer for the utility functions in https://github.com/nikic/php-ast/blob/master/util.php and the stubs in https://github.com/nikic/php-ast/blob/master/ast_stub.php. Though for the latter, it might make more sense to add this extension to https://github.com/JetBrains/phpstorm-stubs, which is whether most tooling sources their stubs from.
@nikic https://github.com/phan/phan/blob/2.6.0/src/Phan/Language/Internal/ConstantDocumentationMap.php#L80 has some stub descriptions already written for the constants declared for php-ast, if you were interested. (this repo has its own descriptions for methods already)
Those were written for Phan's use case (knowing which properties existed in $node->children, from an editor or IDE, in Phan's language server), but may be useful there as well.
