FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[purs ide] Editor mode by kritzcreek · Pull Request #3006 · purescript/purescript · GitHub

[purs ide] Editor mode - #3006

Merged
kritzcreek merged 4 commits into
purescript:masterfrom
kritzcreek:purs-ide-editor-mode
Jul 30, 2017
Merged

[purs ide] Editor mode#3006
kritzcreek merged 4 commits into
purescript:masterfrom
kritzcreek:purs-ide-editor-mode

Conversation

kritzcreek commented Jul 30, 2017
edited
Loading

Copy link
Copy Markdown
Member

Adds a new flag to the server invocation: --editor-mode

When the editor-mode flag is specified at startup the server will not start a file watcher process anymore. Instead it only reloads after successful rebuild commands. This is a lot less fragile than relying on the file system APIs, but will mean that a manual load needs to be triggered after builds that didn't go through purs ide.

Looking forward to get Feedback :) After using psc-ide more intensively the last months it seems like the file watcher is the most fragile part of the system so I'm looking for ways to circumvent that problem.

When the editor-mode flag is specified at startup the server will not start a
file watcher process anymore. Instead it only reloads after succesful rebuild
commands. This is a lot less fragile than relying on the file system APIs, but
will mean that a manual load needs to be triggered after builds that didn't go
through purs ide.

* Also updates the README under psc-ide/ to be a bit more recent
kritzcreek force-pushed the purs-ide-editor-mode branch from 4b0a9c7 to 5757cbc Compare July 30, 2017 14:46

paf31 commented Jul 30, 2017

Copy link
Copy Markdown
Contributor

Is this different from noWatch then?

kritzcreek commented Jul 30, 2017
edited
Loading

Copy link
Copy Markdown
Member Author

Yeah, since it loads the parsed file and generated ExternsFile after rebuilding into the cache. Look at the change inside Rebuild.hs

…f-thread

This also makes the perf measurement mean something
kritzcreek force-pushed the purs-ide-editor-mode branch from 5757cbc to 2606727 Compare July 30, 2017 18:55
kritzcreek merged commit 6e689c4 into purescript:master Jul 30, 2017
kritzcreek deleted the purs-ide-editor-mode branch July 30, 2017 19:55
rhendric mentioned this pull request Aug 23, 2024
5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL