Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Use the editor service npm module directly #40

Merged
merged 2 commits into from
Nov 12, 2016
Merged

Conversation

rictic
Copy link
Contributor

@rictic rictic commented Nov 11, 2016

  • CHANGELOG.md has not been updated, as this is an entirely internal change.

@@ -16,7 +16,8 @@
"dependencies": {
"@types/node": "^6.0.41",
"marked": "^0.3.6",
"polymer-analyzer": "^2.0.0-alpha.15"
"polymer-analyzer": "=2.0.0-alpha.16",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

depending on a specific version here because alpha releases may include breaking changes

@@ -19,8 +19,5 @@
"include": [
"src/**/*.ts",
"custom_typings/main.d.ts"
],
"exclude": [
"node_modules"
]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is implicit, as it's not included in the include section

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While I would assume the same, without these lines the atom-typescript plugin completely locks up my Atom. I am unable to perform edits in the .ts files in this project.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reverted!

I suspect that atom-typescript will eventually fix this bug so we should recheck every once in a while. It's possible that they haven't added support for include, as it was added fairly recently in 2.0, and if it looks like you haven't told typescript what to include (via either include or files then it will just look for **/*.[ts|js], which yeah, is super crazy slow)

Copy link
Contributor

@TimvdLippe TimvdLippe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In general LGTM, but excluding the node_modules for typescript is required to not let Atom lock up during compilation.

@rictic rictic merged commit 4a4e677 into master Nov 12, 2016
@rictic rictic deleted the standalone-editor-service branch November 12, 2016 16:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants