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

Update Analyzer #76

Merged
merged 2 commits into from
Apr 9, 2018
Merged

Update Analyzer #76

merged 2 commits into from
Apr 9, 2018

Conversation

justinfagnani
Copy link
Contributor

No description provided.

@justinfagnani justinfagnani merged commit fcd586d into master Apr 9, 2018
@justinfagnani justinfagnani deleted the analyzer branch April 9, 2018 21:39
rictic added a commit that referenced this pull request Apr 10, 2018
* WIP on updating to the new analyzer apis.

* Drop support for node v4, add node v8.

* Use analyzer alpha.38

* Support node >=6, not >6.
rictic added a commit that referenced this pull request Apr 11, 2018
…protocol. (#76)

* Delete our nonstandard polymer-editor-server

It spoke a custom home-baked JSON protocol. It turned out what we really wanted was the Language Server Protocol

* Remove EditorService abstraction.

* Update changelog.

* Fixup a couple more places in the tests. Delete a noop test.

* Updated comment.

* Update CHANGELOG

* Update changelog.

* Update CHANGELOG again!
aomarks added a commit that referenced this pull request Jul 9, 2018
Hybrid Polymer elements without a LHS assignment now have `Element`
appended to their generated interface name, to match the behavior of the
Closure Polymer Pass
(https://github.com/google/closure-compiler/wiki/Polymer-Pass#element-type-names-for-1xhybrid-call-syntax).
For example, `interface IronRequest` is now `interface
IronRequestElement`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants