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

Declarations update independent of compiler.watchFileSystem #167

Merged

Conversation

opichals
Copy link
Contributor

@opichals opichals commented Mar 3, 2016

Uses watching.compiler.fileTimestamps which is set directly in
the compiler. See https://github.com/webpack/webpack/blob/master/lib/Compiler.js#L105.

Fixes the #155
when using the webpack.OldNodeWatchFileSystem.

Uses watching.compiler.fileTimestamps which is set directly in
the compiler. See https://github.com/webpack/webpack/blob/master/lib/Compiler.js#L105.

Fixes the TypeStrong#155
when using the webpack.OldNodeWatchFileSystem.
@zaqqaz
Copy link

zaqqaz commented Apr 12, 2016

Please merge this pr. It'll helpful.

opichals, thank u, u save me hour 👍

@jbrantly
Copy link
Member

I plan on it. Meant to get it out in the last release but somehow missed it. Will go out in the next.

@zaqqaz
Copy link

zaqqaz commented Apr 12, 2016

Good. It's also solve same problem if use webpack.WatchIgnorePlugin ;)

@benelliott
Copy link

@jbrantly any plans to merge this soon?

@blakeembrey blakeembrey merged commit aff605e into TypeStrong:master Jul 22, 2016
benelliott pushed a commit to benelliott/awesome-typescript-loader that referenced this pull request Aug 2, 2016
s-panferov added a commit to s-panferov/awesome-typescript-loader that referenced this pull request Aug 3, 2016
This was referenced Aug 15, 2016
@pllee
Copy link

pllee commented Oct 7, 2016

Hey can we get an npm publish for this? The latest npm version does not have this fix.

@johnnyreilly
Copy link
Member

Hi @pllee,

I've just started getting to grips with the project (literally yesterday). I'm planning to do an npm publish as soon as I feel it's safe. I need to do some more testing and then I'll try and give it a go.

@pllee
Copy link

pllee commented Oct 7, 2016

Ok thanks.

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.

None yet

7 participants