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

Testing and drive to publish extension with php-language-server using the new parser #404

Closed
roblourens opened this issue Jun 12, 2017 · 15 comments
Assignees
Labels

Comments

@roblourens
Copy link
Contributor

roblourens commented Jun 12, 2017

Where can we post a vsix for testing, where it will get attention and some usage?


Here are the issues that are regressions vs the old parser. Currently, a total of 6 issues:

https://github.com/felixfbecker/php-language-server/issues?q=is%3Aissue+is%3Aopen+label%3ARegression

https://github.com/Microsoft/tolerant-php-parser/issues?q=is%3Aissue+is%3Aopen+label%3Alang-server-regression

I'm not sure whether all of them are blockers?

@felixfbecker
Copy link
Owner

I would do pre-releases here: https://github.com/felixfbecker/vscode-php-intellisense/releases and tweet it

@jens1o
Copy link
Contributor

jens1o commented Jun 13, 2017

Could you ping me up when there's a new one, please?

@felixfbecker
Copy link
Owner

I would recommend you https://sibbell.com/

@jens1o
Copy link
Contributor

jens1o commented Jun 13, 2017

Ah, thanks. Set up. :)

@felixfbecker
Copy link
Owner

@roblourens I am looking into better automatic release of the extension. I have automatic release in CI, but to trigger it I have to push a new tag. It would be possible to generate a vsix for every release and publish it to the GitHub releases page. I could run a Travis cron job every day to release, but it would need to bump the version in package.json like semantic-release does. The hard part is finding out whether the language server got a new release or not, maybe Composer has a command for this.

@staabm
Copy link
Contributor

staabm commented Jun 13, 2017

@felixfbecker
Copy link
Owner

That's great, but can it trigger a Travis build?

@staabm
Copy link
Contributor

staabm commented Jun 13, 2017

I guess you need a little script todo it

https://docs.travis-ci.com/user/triggering-builds/ ;-)

@felixfbecker
Copy link
Owner

And where would that be hosted? Doesn't look like a viable solution to me

@roblourens
Copy link
Contributor Author

I don't know of a better way to automate it - seems like pushing a tag to release when there's a new version we want to test would work for now. Hopefully we won't be in prerelease mode for too long?

What do we need to do to get a new release of the language server?

@roblourens
Copy link
Contributor Author

Also, can you assign this issue to me? We've linked it from the iteration plan to track the task to update the extension with the new parser.

@felixfbecker
Copy link
Owner

I setup Travis to automatically upload vsix packages for every release to the GitHub releases page.
In theory we don't have to do a prerelease at all if it turns out to be pretty stable. Do you think there are any major thing we should fix before going public to prevent issue spam?

@roblourens
Copy link
Contributor Author

The only things I'm aware of are the issues in the linked queries above. I don't even know if they are necessarily blockers - template string support (for find all refs including variables in template strings, etc) is probably the most annoying but I don't know that it would destroy anyone's workflow.

If you just want to go ahead with it and see what happens, I'd be 👍 for that. Or we can try to get one prerelease out there just to see what happens. Up to you.

@roblourens
Copy link
Contributor Author

@felixfbecker Can we get a prerelease out? Waiting for anything?

@ghost
Copy link

ghost commented Jun 20, 2017

this did happen already didn't it?

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

No branches or pull requests

4 participants