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

v0.2.0 #12

Merged
merged 1 commit into from
Jun 2, 2017
Merged

v0.2.0 #12

merged 1 commit into from
Jun 2, 2017

Conversation

piotr-oles
Copy link
Collaborator

Changes:

  • tsconfig.json and tslint.json path are not printed anymore.
  • watch option is not used on 'build' mode
  • Handle case with no options object (new ForkTsCheckerWebpacPlugin())
  • Basic integration tests (along units)

Breaking changes:

  • tslint is not enabled by default - you have to set tslint: true or tslint: './path/to/tslint.json' to enable it.
  • blockEmit option is removed - it choose automatically - blocks always on 'build' mode, never on 'watch' mode.

@piotr-oles piotr-oles merged commit f4bdc3d into master Jun 2, 2017
@piotr-oles piotr-oles deleted the feature/v0.2.0 branch June 2, 2017 21:19
@johnnyreilly
Copy link
Member

This is brilliant, well done! Just curious, what sets this.isWatching to true or false? Read the code but it's not clear to me.

Thanks for all your work by the way - I really appreciate it! I guess I should go through the examples on ts-loader and strip out the block-emit stuff?

@johnnyreilly
Copy link
Member

Made changes with TypeStrong/ts-loader@96a8113

I've also "promoted" fork-ts-checker to near the start of the readme. I figure people are interested in fast builds 😉

@piotr-oles
Copy link
Collaborator Author

isWatching is set to false on event run, and set to true on event watch-run :) Unfortunately, I can't find any flag in webpack`s compiler and compilation object that could tell me if webpack is watching or not :/
Thanks for promotion, I really appreciate this :)

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