many-things / cw-hyperlane

52 stars 26 forks source link

refactor: ts schema #99

Closed byeongsu-hong closed 8 months ago

byeongsu-hong commented 8 months ago

backlog: fix ts-codegen

Currently, this PR only removes existing schema bindings, not generates new one. It's because codegen.ts does not work as i intended, it just throws the errors instead of generating bindings.

We expects the current problem's source is two kind of things and need to fix it.

  1. schema file structure is wrong
  2. ts-codegen is broken...
codecov-commenter commented 8 months ago

Codecov Report

Attention: Patch coverage is 0% with 48 lines in your changes are missing coverage. Please review.

Project coverage is 66.98%. Comparing base (d309bc0) to head (e42da4c).

Files Patch % Lines
packages/schema/src/main.rs 0.00% 48 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #99 +/- ## ========================================== - Coverage 67.82% 66.98% -0.85% ========================================== Files 87 87 Lines 5243 5291 +48 Branches 5243 5291 +48 ========================================== - Hits 3556 3544 -12 - Misses 1302 1360 +58 - Partials 385 387 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

github-actions[bot] commented 8 months ago

🤖 Pull request artifacts

file commit
wasm_codes.zip e42da4cb9f39b13f178128441fb57a6f08ff1073
byeongsu-hong commented 8 months ago

@nambrot Thanks for your review, and i left the brief description about current problem.