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

swim-build reports errors in log but returns a success (0) exit code. #132

Open
jcustenborder opened this issue Sep 12, 2023 · 2 comments · Fixed by #136
Open

swim-build reports errors in log but returns a success (0) exit code. #132

jcustenborder opened this issue Sep 12, 2023 · 2 comments · Fixed by #136
Assignees
Labels
C-bug Category: bug

Comments

@jcustenborder
Copy link
Contributor

I'm seeing compilation failures when running npx swim-build but the exit code is 0, so scripts see it as successful.

npx swim-build
compiling @swim/util:main
compiled @swim/util:main in 20ms

compiling @swim/util:test
compiled @swim/util:test in 3ms

compiling @swim/codec:main
compiled @swim/codec:main in 23ms

compiling @swim/codec:test
compiled @swim/codec:test in 13ms

compiling @swim/component:main
compiled @swim/component:main in 16ms

compiling @swim/component:test
compiled @swim/component:test in 14ms

compiling @swim/collections:main
compiled @swim/collections:main in 21ms

compiling @swim/collections:test
compiled @swim/collections:test in 21ms

compiling @swim/constraint:main
compiled @swim/constraint:main in 25ms

compiling @swim/constraint:test
compiled @swim/constraint:test in 25ms

compiling @swim/structure:main
compiled @swim/structure:main in 30ms

compiling @swim/structure:test
compiled @swim/structure:test in 22ms

compiling @swim/recon:main
compiled @swim/recon:main in 39ms

compiling @swim/recon:test
compiled @swim/recon:test in 34ms

compiling @swim/uri:main
compiled @swim/uri:main in 38ms

compiling @swim/uri:test
compiled @swim/uri:test in 32ms

compiling @swim/math:main
info: 7053
   --> /Users/jeremy/source/nstream/swimos/swim/swim-js/swim-runtime/swim-core/swim-math/src/main/transform/Transform.ts:222:50
    |
222 |       return Transform.fromCssTransformComponent(value[0]!);
    |                                                  ^^^^^^^^ Element implicitly has an 'any' type because expression of type '0' can't be used to index type 'CSSTransformValue'.
info: 7053
   --> /Users/jeremy/source/nstream/swimos/swim/swim-js/swim-runtime/swim-core/swim-math/src/main/transform/Transform.ts:226:61
    |
226 |         transforms[i] = Transform.fromCssTransformComponent(value[i]!);
    |                                                             ^^^^^^^^ Element implicitly has an 'any' type because expression of type 'number' can't be used to index type 'CSSTransformValue'.
info: 2339
   --> /Users/jeremy/source/nstream/swimos/swim/swim-js/swim-runtime/swim-core/swim-math/src/main/transform/ScaleTransform.ts:150:27
    |
150 |             : component.x.to("number").value;
    |                           ^^ Property 'to' does not exist on type 'never'.
info: 2339
   --> /Users/jeremy/source/nstream/swimos/swim/swim-js/swim-runtime/swim-core/swim-math/src/main/transform/ScaleTransform.ts:153:27
    |
153 |             : component.y.to("number").value;
    |                           ^^ Property 'to' does not exist on type 'never'.
failed to compile @swim/math:main

unable to build @swim/math:test

unable to build @swim/geo:main

unable to build @swim/geo:test

compiling @swim/time:main
compiled @swim/time:main in 47ms

compiling @swim/time:test
compiled @swim/time:test in 34ms

unable to build @swim/core:main

compiling @swim/warp:main
compiled @swim/warp:main in 50ms

compiling @swim/warp:test
compiled @swim/warp:test in 45ms

compiling @swim/args:main
compiled @swim/args:main in 21ms

compiling @swim/args:test
compiled @swim/args:test in 19ms

compiling @swim/client:worker
compiled @swim/client:worker in 56ms

compiling @swim/client:webworker
compiled @swim/client:webworker in 58ms

compiling @swim/client:main
compiled @swim/client:main in 59ms

compiling @swim/client:cli
compiled @swim/client:cli in 60ms

compiling @swim/client:test
info: 2345
  --> /Users/jeremy/source/nstream/swimos/swim/swim-js/swim-runtime/swim-host/swim-client/src/test/MockServer.ts:59:23
   |
59 |     this.socket!.send(data);
   |                       ^^^^ Argument of type 'unknown' is not assignable to parameter of type 'BufferLike'.
failed to compile @swim/client:test

unable to build @swim/host:main

unable to build @swim/runtime:main

compiling @swim/sys:main
compiled @swim/sys:main in 28ms

compiling @swim/unit:main
compiled @swim/unit:main in 21ms

compiling @swim/unit:cli
compiled @swim/unit:cli in 19ms

compiling @swim/build:main
compiled @swim/build:main in 30ms

compiling @swim/build:cli
compiled @swim/build:cli in 24ms

compiling @swim/macro:main
compiled @swim/macro:main in 45ms

compiling @swim/macro:cli
compiled @swim/macro:cli in 43ms

compiling @swim/macro:test
compiled @swim/macro:test in 46ms

unable to build @swim/model:main

unable to build @swim/style:main

unable to build @swim/style:test

unable to build @swim/theme:main

unable to build @swim/theme:test

unable to build @swim/view:main

unable to build @swim/dom:main

unable to build @swim/graphics:main

unable to build @swim/controller:main

unable to build @swim/ui:main

unable to build @swim/button:main

unable to build @swim/toolbar:main

unable to build @swim/token:main

unable to build @swim/table:main

unable to build @swim/window:main

unable to build @swim/sheet:main

unable to build @swim/panel:main

unable to build @swim/ux:main

unable to build @swim/gauge:main

unable to build @swim/pie:main

unable to build @swim/chart:main

unable to build @swim/vis:main

unable to build @swim/map:main

unable to build @swim/mapbox:main

unable to build @swim/leaflet:main

unable to build @swim/googlemap:main

unable to build @swim/esrimap:main

unable to build @swim/maps:main

unable to build @swim/toolkit:main
@jcustenborder jcustenborder added the C-bug Category: bug label Sep 12, 2023
@ajay-gov ajay-gov self-assigned this Sep 12, 2023
@ajay-gov
Copy link
Member

The compilation error with swim-client should be fixed with this commit

Can't reproduce the compilation error you are getting with the swim-math library. What is the node and typescript version you are using @jcustenborder

@ajay-gov ajay-gov linked a pull request Sep 18, 2023 that will close this issue
@ajay-gov
Copy link
Member

Reopening, the commit only fixes the compilation error. This bug is about returning a non-zero value while running npx swim-build

@ajay-gov ajay-gov reopened this Sep 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants