Skip to content

refactor: make type option readable and refactor workflow #718

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 6, 2024
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
55 changes: 17 additions & 38 deletions .github/workflows/build-templates.yml
Original file line number Diff line number Diff line change
@@ -30,44 +30,30 @@ jobs:
- ubuntu-latest
- macos-14 # macos latest defaults to macos 12 at the moment.
type:
- module-legacy
- module-new
- view-legacy
- view-new
- turbo-module
- fabric-view
- legacy-module
- legacy-view
language:
- kotlin-objc
- kotlin-swift
arch:
- auto
- cpp
exclude:
- type: module-new
- type: turbo-module
language: kotlin-swift
- type: view-new
- type: fabric-view
language: kotlin-swift
- type: fabric-view
language: cpp
- type: legacy-module
language: cpp
include:
- os: ubuntu-latest
type: library
language: js
arch: auto
- os: ubuntu-latest
type: module-legacy
language: cpp
arch: auto
- os: ubuntu-latest
type: module-new
language: cpp
arch: auto
- os: macos-14
type: module-legacy
language: cpp
arch: auto
- os: macos-14
type: module-new
language: cpp
arch: auto

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}-${{ matrix.os }}-${{ matrix.type }}-${{ matrix.language }}-${{ matrix.arch }}
group: ${{ github.workflow }}-${{ github.ref }}-${{ matrix.os }}-${{ matrix.type }}-${{ matrix.language }}
cancel-in-progress: true

runs-on: ${{ matrix.os }}
@@ -79,20 +65,13 @@ jobs:
- name: Setup
uses: ./.github/actions/setup

- name: Set environment variables
run: |
if [[ "${{ matrix.arch }}" == "new" ]]; then
echo "RCT_NEW_ARCH_ENABLED=1" >> $GITHUB_ENV
echo "ORG_GRADLE_PROJECT_newArchEnabled=true" >> $GITHUB_ENV
fi
- name: Build package
run: |
yarn workspace create-react-native-library prepare
- name: Get working directory
run: |
echo "work_dir=${{ matrix.os }}-${{ matrix.type }}-${{ matrix.language }}-${{ matrix.arch }}" >> $GITHUB_ENV
echo "work_dir=${{ matrix.os }}-${{ matrix.type }}-${{ matrix.language }}" >> $GITHUB_ENV
- name: Create library
run: |
@@ -142,14 +121,14 @@ jobs:
run: |
# Build Android for only some matrices to skip redundant builds
if [[ ${{ matrix.os }} == ubuntu-latest ]]; then
if [[ ${{ matrix.type }} == view-* && ${{ matrix.language }} == *-objc ]] || [[ ${{ matrix.type }} == module-* && ${{ matrix.language }} == *-objc ]] || [[ ${{ matrix.type }} == module-* && ${{ matrix.language }} == cpp ]]; then
if [[ ${{ matrix.type }} == *-view && ${{ matrix.language }} == *-objc ]] || [[ ${{ matrix.type }} == *-module && ${{ matrix.language }} == *-objc ]] || [[ ${{ matrix.type }} == *-module && ${{ matrix.language }} == cpp ]]; then
echo "android_build=1" >> $GITHUB_ENV
fi
fi
# Build iOS for only some matrices to skip redundant builds
if [[ ${{ matrix.os }} == macos-14 ]]; then
if [[ ${{ matrix.type }} == view-* && ${{ matrix.language }} == kotlin-* ]] || [[ ${{ matrix.type }} == module-* && ${{ matrix.language }} == kotlin-* ]] || [[ ${{ matrix.type }} == module-* && ${{ matrix.language }} == cpp ]]; then
if [[ ${{ matrix.type }} == *-view && ${{ matrix.language }} == kotlin-* ]] || [[ ${{ matrix.type }} == *-module && ${{ matrix.language }} == kotlin-* ]] || [[ ${{ matrix.type }} == *-module && ${{ matrix.language }} == cpp ]]; then
echo "ios_build=1" >> $GITHUB_ENV
fi
fi
@@ -160,9 +139,9 @@ jobs:
with:
path: |
${{ env.work_dir }}/.turbo
key: ${{ runner.os }}-library-turborepo-${{ matrix.type }}-${{ matrix.language }}-${{ matrix.arch }}-${{ hashFiles(format('{0}/yarn.lock', env.work_dir)) }}
key: ${{ runner.os }}-library-turborepo-${{ matrix.type }}-${{ matrix.language }}-${{ hashFiles(format('{0}/yarn.lock', env.work_dir)) }}
restore-keys: |
${{ runner.os }}-library-turborepo-${{ matrix.type }}-${{ matrix.language }}-${{ matrix.arch }}-
${{ runner.os }}-library-turborepo-${{ matrix.type }}-${{ matrix.language }}-
- name: Check turborepo cache
if: env.android_build == 1 || env.ios_build == 1
2 changes: 1 addition & 1 deletion packages/create-react-native-library/src/index.ts
Original file line number Diff line number Diff line change
@@ -121,7 +121,7 @@ async function create(_argv: yargs.Arguments<Args>) {
: devDependencies;
}

if (config.example === 'vanilla' && config.project.arch !== 'legacy') {
if (config.example === 'vanilla' && config.project.arch === 'new') {
addCodegenBuildScript(folder);
}

22 changes: 11 additions & 11 deletions packages/create-react-native-library/src/input.ts
Original file line number Diff line number Diff line change
@@ -21,10 +21,10 @@ export type ArgName =
export type ProjectLanguages = 'kotlin-objc' | 'kotlin-swift' | 'cpp' | 'js';

export type ProjectType =
| 'module-new'
| 'view-new'
| 'module-legacy'
| 'view-legacy'
| 'turbo-module'
| 'fabric-view'
| 'legacy-module'
| 'legacy-view'
| 'library';

const LANGUAGE_CHOICES: {
@@ -35,17 +35,17 @@ const LANGUAGE_CHOICES: {
{
title: 'Kotlin & Objective-C',
value: 'kotlin-objc',
types: ['module-new', 'view-new', 'module-legacy', 'view-legacy'],
types: ['turbo-module', 'fabric-view', 'legacy-module', 'legacy-view'],
},
{
title: 'Kotlin & Swift',
value: 'kotlin-swift',
types: ['module-legacy', 'view-legacy'],
types: ['legacy-module', 'legacy-view'],
},
{
title: 'C++ for Android & iOS',
value: 'cpp',
types: ['module-new', 'module-legacy'],
types: ['turbo-module', 'legacy-module'],
},
{
title: 'JavaScript for Android, iOS & Web',
@@ -86,22 +86,22 @@ const TYPE_CHOICES: {
}[] = [
{
title: 'Turbo module',
value: 'module-new',
value: 'turbo-module',
description: 'integration for native APIs to JS',
},
{
title: 'Fabric view',
value: 'view-new',
value: 'fabric-view',
description: 'integration for native views to JS',
},
{
title: 'Legacy Native module',
value: 'module-legacy',
value: 'legacy-module',
description: 'bridge for native APIs to JS (old architecture)',
},
{
title: 'Legacy Native view',
value: 'view-legacy',
value: 'legacy-view',
description: 'bridge for native views to JS (old architecture)',
},
{
6 changes: 3 additions & 3 deletions packages/create-react-native-library/src/template.ts
Original file line number Diff line number Diff line change
@@ -104,7 +104,7 @@ export function generateTemplateConfiguration({
const { slug, languages, type } = answers;

const arch =
type === 'module-legacy' || type === 'view-legacy' ? 'legacy' : 'new';
type === 'legacy-module' || type === 'legacy-view' ? 'legacy' : 'new';

const project = slug.replace(/^(react-native-|@[^/]+\/)/, '');
let namespace: string | undefined;
@@ -144,8 +144,8 @@ export function generateTemplateConfiguration({
arch,
cpp: languages === 'cpp',
swift: languages === 'kotlin-swift',
view: answers.type.startsWith('view'),
module: answers.type.startsWith('module'),
view: answers.type.endsWith('-view'),
module: answers.type.endsWith('-module'),
},
author: {
name: answers.authorName,
Original file line number Diff line number Diff line change
@@ -3,7 +3,7 @@
"version": "0.0.0",
"description": "<%- project.description %>",
"main": "src/index",
<% if (project.arch !== 'legacy') { -%>
<% if (project.arch === 'new') { -%>
"codegenConfig": {
"name": "RN<%- project.name -%><%- project.view ? 'View': '' -%>Spec",
"type": <%- project.view ? '"all"': '"modules"' %>,
Original file line number Diff line number Diff line change
@@ -172,7 +172,7 @@
"source": "src",
"output": "lib",
"targets": [
<% if (project.arch !== 'legacy') { -%>
<% if (project.arch === 'new') { -%>
"codegen",
<% } -%>
[
@@ -194,7 +194,7 @@
}
]
]
<% if (project.arch !== 'legacy') { -%>
<% if (project.arch === 'new') { -%>
},
"codegenConfig": {
"name": "RN<%- project.name -%><%- project.view ? 'View': '' -%>Spec",
Original file line number Diff line number Diff line change
@@ -49,7 +49,7 @@ To run the example app on iOS:
yarn example ios
```

<% if (project.arch !== 'legacy') { -%>
<% if (project.arch === 'new') { -%>
To confirm that the app is running with the new architecture, you can check the Metro logs for a message like this:

```sh