[native] Install CommUtils JSI module
ClosedPublic
Actions

Authored by bartek on Apr 14 2023, 12:36 PM.

Details

Reviewers

atul
marcin
kamil

Commits

rCOMM4859024da48d: [native] Install CommUtils JSI module

Summary

Added code to install the CommUtilsModule JSI module for both Android and iOS

Depends on D7451

Test Plan

Both platforms compile.
Replaced the dummy jsi::Value::undefined() returns in CommUtilsModule with e.g. jsi::Value(42) and verified that these are callable from JS.
CommCoreModule remains unaffected and still works the same way as previously

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Apr 14 2023, 12:36 PM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptApr 14 2023, 12:36 PM

Harbormaster failed remote builds in B18369: Diff 25163!Apr 14 2023, 12:54 PM

Rebase

Harbormaster completed remote builds in B18371: Diff 25165.Apr 14 2023, 1:15 PM

bartek published this revision for review.Apr 14 2023, 3:01 PM

Don't have a ton of experience here but makes sense, would be good to amend test plan to make sure that we can call functions in CommCoreModule as before

This revision is now accepted and ready to land.Apr 17 2023, 1:36 PM

In D7452#221895, @atul wrote:

Don't have a ton of experience here but makes sense, would be good to amend test plan to make sure that we can call functions in CommCoreModule as before

Checked, works as expected

bartek added a child revision: D7485: [native][utils-module] Implement file r/w operations.Apr 18 2023, 3:07 PM

Rebase before landing

Harbormaster completed remote builds in B18601: Diff 25478.Apr 21 2023, 7:59 AM

Closed by commit rCOMM4859024da48d: [native] Install CommUtils JSI module. · Explain WhyApr 21 2023, 9:20 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMM4859024da48d: [native] Install CommUtils JSI module.