8000 WIP! chore: use rangutopia for building wallets by yeager-eren · Pull Request #1223 · rango-exchange/rango-client · GitHub
[go: up one dir, main page]

Skip to content

Conversation

yeager-eren
Copy link
Collaborator
@yeager-eren yeager-eren commented Aug 1, 2025

Summary

Replace build/command.js with rangutopia scripts.

Fixes # (issue)

How did you test this change?

We should test in two phase, the first one using local linking (before rangutopia is available on NPM) and second when it's published.

Phase 1: local

First run:

yarn

We only want to test two packages at this phase, the output of script and rangutopia should be same:

cd wallets/core/ 
yarn clean
yarn build2
mv dist dist2
yarn build
diff -qr dist dist2

do same for wallets/provider-ledger.
Note: not sure why hashes are different but content seems to be same. you can check them using this tool.

You can also run from root:

yarn workspace @rango-dev/wallets-core run build

Phase 2: npm

tba

Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • Implemented a user interface (UI) change, referencing our Figma design to ensure pixel-perfect precision.

@yeager-eren yeager-eren force-pushed the chore/use-rangutopia branch from 87794b0 to 2d43301 Compare August 1, 2025 17:58
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.

1 participant
0