infinyon / infinyon/node-bindgen

Support nodejs-mobile in nj-cli build

Open
#163 3 comments 2 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

build CI documentation platform/mobile
Dominant language
Rust
Stars
587
Forks
43
PR merge metrics
No merged PRs in 30d

Description

With a few tweaks, node-bindgen actually supports iOS and Android via nodejs-mobile.

I don't have time available and knowledge of this codebase enough to submit a PR, but if you (maintainers) want to support nodejs-mobile, then it's a low-hanging fruit since you can just follow the steps I implemented here:

https://github.com/ssb-ngi-pointer/ssb-validate2-rsjs-node/blob/289a6bd92d651d6d97cf07f89d6325ba82686d80/install.js#L33-L44

Thanks for node-bindgen!

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Read the referenced install.js lines 33-44 to understand the nodejs-mobile setup, then inspect the nj-cli build path in node-bindgen. Verify that the build supports nodejs-mobile on both iOS and Android and that the resulting modules can be installed and used.

Written by the indexing model from the issue text.

Assessment

Tech stack
nodejs, rust
Domain
build-system, cli, mobile
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.