Skip to content

Topic/psc ide #77

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

Merged
merged 4 commits into from
Nov 19, 2016
Merged

Topic/psc ide #77

merged 4 commits into from
Nov 19, 2016

Conversation

ethul
Copy link
Owner

@ethul ethul commented Nov 19, 2016

No description provided.

The objective of enabling psc-ide is to obtain feedback immediately from
the PureScript compiler. When the loader triggers an error to webpack,
the bundle is invalidated and dependencies of the PureScript file that
caused the error will be passed through the loader on the next
successful rebuild.

However, this behaviour adds time to the feedback loop. In order to
reduce the time of the PureScript compiler feedback loop when in
development, this commit always resolves the rebuild process with a
success. Errors and warnings are still emitted, but the webpack bundle
is not invalidated. This means that the loader will only be run for the
current file being editted and the developer gets immediate feedback on
success or failure when psc-ide is enabled.

Resolves #76
Instead of forcing a recompile on an unknown value and on unknown module
imports, opt to only recompile for unknown module imports. An unknown
value may be a typo. Having a faster feedback loop in these cases seems
ideal.
@ethul ethul merged commit 0aa15fb into master Nov 19, 2016
@ethul ethul deleted the topic/psc-ide branch November 19, 2016 19:09
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.

1 participant