Skip to content
This repository was archived by the owner on Dec 1, 2019. It is now read-only.

Apply https://github.com/TypeStrong/ts-loader/pull/167 #204

Merged
s-panferov merged 1 commit intos-panferov:masterfrom
benelliott:master
Aug 3, 2016
Merged

Apply https://github.com/TypeStrong/ts-loader/pull/167 #204
s-panferov merged 1 commit intos-panferov:masterfrom
benelliott:master

Conversation

@benelliott
Copy link
Copy Markdown
Contributor

to improve watch stability when using OldWatchPlugin or WatchIgnorePlugin.

See TypeStrong/ts-loader#155

@s-panferov
Copy link
Copy Markdown
Owner

Thanks!

@s-panferov s-panferov merged commit b18ab9a into s-panferov:master Aug 3, 2016
@d-ph
Copy link
Copy Markdown

d-ph commented Aug 14, 2016

Hello @s-panferov . Could you release a 2.1.2 version with this fix? Thanks.

@s-panferov
Copy link
Copy Markdown
Owner

@d-ph landed in 2.2.0

@d-ph
Copy link
Copy Markdown

d-ph commented Aug 15, 2016

Legend. Thanks, mate. 👍

@guncha
Copy link
Copy Markdown
Contributor

guncha commented Oct 24, 2016

This is causing a 3-5x degradation in re-build performance for us 😞

After some debugging it seems that the problem is that watching.compiler.fileTimestamps seems to contain mtimes for about 20k files instead of the ones that actually changed.

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.

5 participants