Flow-typed: Set up AppVeyor

Created on 16 Aug 2016  路  3Comments  路  Source: flow-typed/flow-typed

Need to make sure that the cli doesn't break on Windows.

enhancement

Most helpful comment

@szarouski: I pushed a fix for this yesterday, but haven't published it yet: https://github.com/flowtype/flow-typed/commit/567032eb631e873d457bf5ffe51b99c02a44e9b6

Will do this very soon: I'm trying to get a few more PRs merged before publishing the next beta release.

All 3 comments

Would do that, need flowtype organisation access though.

Hi, Current version is not working on windows:

C:\dev\bugsnag-sourcemaps>flow-typed install -f 0.30 [email protected]
C:\Users\szarouski\AppData\Roaming\npm\node_modules\flow-typed\dist\commands\run
Tests.js:45
      throw new Error("Unsupported os.type()! " + _node.os.type());
      ^

Error: Unsupported os.type()! Windows_NT
    at C:\Users\szarouski\AppData\Roaming\npm\node_modules\flow-typed\dist\comma
nds\runTests.js:45:13
    at Object.<anonymous> (C:\Users\szarouski\AppData\Roaming\npm\node_modules\f
low-typed\dist\commands\runTests.js:47:2)
    at Module._compile (module.js:556:32)
    at Object.Module._extensions..js (module.js:565:10)
    at Module.load (module.js:473:32)
    at tryModuleLoad (module.js:432:12)
    at Function.Module._load (module.js:424:3)
    at Module.require (module.js:483:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (C:\Users\szarouski\AppData\Roaming\npm\node_modules\f
low-typed\dist\cli.js:22:17)

Should I create a new issue or this thread is good enough?
Thank you.


update, if I comment out runTests from cli.js, I can get further.

@szarouski: I pushed a fix for this yesterday, but haven't published it yet: https://github.com/flowtype/flow-typed/commit/567032eb631e873d457bf5ffe51b99c02a44e9b6

Will do this very soon: I'm trying to get a few more PRs merged before publishing the next beta release.

Was this page helpful?
0 / 5 - 0 ratings