Skip to content

refactor(crnl): split the index file #688

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 24 commits into from
Nov 29, 2024
Merged

refactor(crnl): split the index file #688

merged 24 commits into from
Nov 29, 2024

Conversation

atlj
Copy link
Collaborator

@atlj atlj commented Nov 16, 2024

Summary

The previous index.ts file of create-react-native-library was too long and complicated to maintain. This PR splits the file a bit

Test plan

  1. We have to make sure that the behavior hasn't changed at all. CI pipeline is the best way to check this.

@atlj atlj changed the title refactor: crnl index file refactor(crnl): split the index file Nov 16, 2024
import fs from 'fs-extra';
import type { ExampleApp } from '../input';

export async function getDependencyVersionsFromExample(
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
export async function getDependencyVersionsFromExample(
export async function getDependencyVersionsFromExampleApp(

folder: string,
exampleAppType: ExampleApp
) {
// Set `react` and `react-native` versions of root `package.json` from example `package.json`
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
// Set `react` and `react-native` versions of root `package.json` from example `package.json`

@atlj atlj force-pushed the @atlj/refactor-crnl-main branch from ef29e22 to ffd2eed Compare November 29, 2024 09:12
@atlj atlj force-pushed the @atlj/refactor-crnl-main branch from ffd2eed to ffc4517 Compare November 29, 2024 09:15
@atlj atlj merged commit e969c89 into main Nov 29, 2024
21 of 29 checks passed
@atlj atlj deleted the @atlj/refactor-crnl-main branch November 29, 2024 09:38
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