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

Release 3.0 #267

Closed
13 of 14 tasks
s-panferov opened this issue Nov 3, 2016 · 23 comments
Closed
13 of 14 tasks

Release 3.0 #267

s-panferov opened this issue Nov 3, 2016 · 23 comments
Milestone

Comments

@s-panferov
Copy link
Owner

s-panferov commented Nov 3, 2016

  • Refactor
  • Always fork a child process for speed
  • Remove all custom resolution logic
  • Support typescript 2.0.6 and the latest webpack@beta
  • Align configuration options with ts-loader
  • Test on typescript@next
  • Test on Windows
  • Receive feedback
  • Update README
  • Consider to port separate-process-checker to ts-loader, Joining forces with ts-loader? #266

Blockers:

@Ciantic
Copy link

Ciantic commented Nov 4, 2016

I tested on windows, seems to work so far. I'll post here if I encounter something with my Windows setup.

@s-panferov
Copy link
Owner Author

@Ciantic thanks! I appreciate that.

@endel
Copy link

endel commented Nov 9, 2016

I just tried out v 3.0.0.beta.0, and I'm having this error:

Error: Cannot find module 'enhanced-resolve/lib/getInnerRequest'

Do you have any clue? Thanks!

@dsifford
Copy link

dsifford commented Nov 9, 2016

Just updated and it looks like ForkCheckerPlugin is now either broken, or deprecated.

Also I'm getting this error:

(node:18815) Warning: Possible EventEmitter memory leak detected. 11 exit listeners added. Use emitter.setMaxListeners() to increaselimit

Also: Even after it crashes, the process is still open. In other words, I have to press cmd-c just to get the process to exit.

@niieani
Copy link

niieani commented Nov 15, 2016

It would be good to tackle this issue before 3.0: #129.

@cortopy
Copy link

cortopy commented Nov 18, 2016

beta2 works really good for me on Linux. In fact, it just solved an issue I've had with tslint output for a long time #273

@jefbarn
Copy link

jefbarn commented Nov 20, 2016

beta.3 using typescript 2.1.1 just hangs after showing
[at-loader] Using typescript@2.1.1 from typescript and "tsconfig.json" from C:\projects\dayapp/tsconfig.json

@s-panferov s-panferov modified the milestone: 3.0 Nov 23, 2016
@vascofernandes
Copy link

same problem as jefbarn but for TS 2.0.10

@s-panferov
Copy link
Owner Author

@vascofernandes do you have some example to reproduce?

@LennyThompson
Copy link

ATL 3.0.0-beta.3 is ignoring exclude configuration from webpack - using filter /.(spec|e2e).ts/ still compiles the spec files. You can test this with the angular2-webpack-starter by updating ATL to latest and removing the ForkCheckerPlugin reference... Reverting to 2.2.1 fixes the problem.

@niieani
Copy link

niieani commented Nov 24, 2016

@LennyThompson I think you should ignore those files from within tsconfig.json. If you need, you can create a second tsconfig.json for the production build that doesn't include these spec files.

@LennyThompson
Copy link

LennyThompson commented Nov 24, 2016 via email

@niieani
Copy link

niieani commented Nov 24, 2016

Not sure what's the designed behavior for exclude configuration @LennyThompson, but ATL should follow the exclude list in tsconfig in any case.

@s-panferov
Copy link
Owner Author

s-panferov commented Nov 25, 2016

@LennyThompson @niieani we have two different conceptions here:

  1. "TypeScript program" — it's a scope of files which will be type-checked together. This scope is controlled by tsconfig.json.

  2. Webpack Bundle — it's a scope of files which will be transpiled and included to a result bundle. This scope is controlled by webpack.config.js and its exclude rules.

@niieani
Copy link

niieani commented Nov 25, 2016

@s-panferov yeah, this is how I understood it. Good points to add to the README, though ;)

@LennyThompson
Copy link

LennyThompson commented Nov 25, 2016 via email

@lhecker
Copy link

lhecker commented Nov 26, 2016

@s-panferov When using the webpack@2.1.0-beta.27 I'm constantly getting the following:

Typescript checker was exited with non-zero error code

Strangely if I use the webpack-dev-server@2.1.0-beta.12 it works fine though. (Using the same webpack.config.js!)

Any idea what could cause or how I should debug this?

@jefbarn
Copy link

jefbarn commented Dec 1, 2016

Just wanted to let you know that 3.0 beta.9 is working again with my project. Thanks!

@pselden
Copy link

pselden commented Dec 6, 2016

Has anyone seen this issue before (with beta.9):

screen shot 2016-12-06 at 11 00 25 am

This is in a very large project that uses 2 instances of ATL (and unfortunately I can't post the code). If no one has encountered an issue like this before I'll try to produce a minimal sample project that exhibits it.

Also -- the error repeats it self around 500 times.

More info: it doesn't always happen, sometimes it happens on first build, but mostly happens on rebuild during a watch.

@cortopy
Copy link

cortopy commented Dec 6, 2016

@pselden I've had that many times. It means lodash definitions are included more than once in your parsing. But I wouldn't say it has to do with v3 per se

@pselden
Copy link

pselden commented Dec 6, 2016

#267 (comment) -- thanks, I just never saw it before until I updated to 3.x beta.

How would I go about fixing that then -- making sure there's only one lodash in my node_modules? One of the reasons I thought it might be an issue with v3 was that it only happens during a rebuild.

@cortopy
Copy link

cortopy commented Dec 6, 2016

Whenever it's happened to me, there was always a lot of trial and error. It used to happen a lot with my npm-linked modules, but new versions of TS have fixed that. Try with different options in tsconfig too (exclude the right folders, include only what's needed, etc.)

@pselden
Copy link

pselden commented Dec 15, 2016

Now with ATL .17 beta I'm seeing a lot of "unknown error"

{ seq: 348, success: true, payload: null } // logged out res
Unknown message:  null

And then the build fails -- while in previous versions it would succeed. This does not happen when running in watch mode.

Any ideas where to look?

Note: "tsc" works correctly.

Another edit: I think it is similar to this other issue I see: #323 -- although I am using multiple instances of at-loader by using the 'instance' property (instead of multiple configs)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests