Skip to content

Conversation

@okwasniewski
Copy link
Member

Overview

This PR introduces a big package overhaul.

First of all there is no more JavaScript, everything is migrated to TypeScript, build scripts are using react-native-builder-bob. The example is now using latest version of React Native.

Test Plan

This is only bumping example and changing the build process. Improvements like new architecture support are coming in next PRs

@okwasniewski okwasniewski force-pushed the feat/package-overhaul branch 2 times, most recently from 58c04c3 to 467e679 Compare October 6, 2025 08:18
@okwasniewski okwasniewski force-pushed the feat/package-overhaul branch from 467e679 to d9b08ac Compare October 6, 2025 08:30
@@ -0,0 +1,42 @@
# EXAMPLE USAGE:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mistake? This is a sample lefthook config and not actual config

@@ -0,0 +1,71 @@
import { type ReactNode, useEffect, useRef, useState } from 'react';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I prefer .native.tsx and .tsx instead because .web isn't supported in all bundlers and needs additional config.

"peerDependencies": {
"react": ">=16",
"react-native": ">=0.57"
"modern-screenshot": "*",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason it can't be a regular dep?

- name: Setup Node.js
uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
with:
node-version-file: .nvmrc
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing .nvmrc?

@satya164 satya164 changed the title feat: package overhaul MOB-862 package overhaul Oct 6, 2025
@satya164 satya164 changed the title MOB-862 package overhaul refactor: package overhaul Oct 6, 2025
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.

3 participants