Skip to content

Conversation

@NorbertKlockiewicz
Copy link
Contributor

@NorbertKlockiewicz NorbertKlockiewicz commented Sep 22, 2025

Description

Clean up repository structure and update library paths

Summary:

  • Remove generated iOS/Android app directories from all demo apps (computer-vision, llm, speech-to-text, text-embeddings)
  • Update .gitignore to exclude apps/*/ios/ and apps/*/android/directories
  • Move native library binaries from packages/react-native-executorch/ios/libs/ to packages/react-native-executorch/third-party/ios/libs
  • Update podspec to reference new library paths in third-party directory
  • Remove executorch git submodule

Introduces a breaking change?

  • Yes
  • No

Type of change

  • Bug fix (change which fixes an issue)
  • New feature (change which adds functionality)
  • Documentation update (improves or adds clarity to existing documentation)
  • Other (chores, tests, code style improvements etc.)

Tested on

  • iOS
  • Android

Testing instructions

Screenshots

Related issues

Checklist

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation accordingly
  • My changes generate no new warnings

Additional notes

@NorbertKlockiewicz NorbertKlockiewicz changed the title feat: repository cleared chore: repository cleared Sep 22, 2025
Copy link
Contributor Author

@NorbertKlockiewicz NorbertKlockiewicz left a comment

Choose a reason for hiding this comment

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

Add increased memory capability to llm example app expo.json

@NorbertKlockiewicz NorbertKlockiewicz merged commit 9b08acf into main Oct 1, 2025
4 checks passed
@NorbertKlockiewicz NorbertKlockiewicz deleted the @nk/clean-repository branch October 1, 2025 13:29
NorbertKlockiewicz added a commit that referenced this pull request Oct 1, 2025
  Clean up repository structure and update library paths

  Summary:
- Remove generated iOS/Android app directories from all demo apps
(computer-vision, llm, speech-to-text, text-embeddings)
- Update .gitignore to exclude `apps/*/ios/ `and
`apps/*/android/`directories
- Move native library binaries from
`packages/react-native-executorch/ios/libs/` to
`packages/react-native-executorch/third-party/ios/libs`
- Update podspec to reference new library paths in third-party directory
  - Remove executorch git submodule

- [ ] Yes
- [x] No

- [ ] Bug fix (change which fixes an issue)
- [ ] New feature (change which adds functionality)
- [ ] Documentation update (improves or adds clarity to existing
documentation)
- [x] Other (chores, tests, code style improvements etc.)

- [x] iOS
- [x] Android

<!-- Provide step-by-step instructions on how to test your changes.
Include setup details if necessary. -->

<!-- Add screenshots here, if applicable -->

<!-- Link related issues here using #issue-number -->

- [x] I have performed a self-review of my code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have updated the documentation accordingly
- [x] My changes generate no new warnings

<!-- Include any additional information, assumptions, or context that
reviewers might need to understand this PR. -->
KnextKoder pushed a commit to Synkhiv/react-native-executorch that referenced this pull request Nov 7, 2025
## Description

  Clean up repository structure and update library paths

  Summary:
- Remove generated iOS/Android app directories from all demo apps
(computer-vision, llm, speech-to-text, text-embeddings)
- Update .gitignore to exclude `apps/*/ios/ `and
`apps/*/android/`directories
- Move native library binaries from
`packages/react-native-executorch/ios/libs/` to
`packages/react-native-executorch/third-party/ios/libs`
- Update podspec to reference new library paths in third-party directory
  - Remove executorch git submodule

### Introduces a breaking change?

- [ ] Yes
- [x] No

### Type of change

- [ ] Bug fix (change which fixes an issue)
- [ ] New feature (change which adds functionality)
- [ ] Documentation update (improves or adds clarity to existing
documentation)
- [x] Other (chores, tests, code style improvements etc.)

### Tested on

- [x] iOS
- [x] Android

### Testing instructions

<!-- Provide step-by-step instructions on how to test your changes.
Include setup details if necessary. -->

### Screenshots

<!-- Add screenshots here, if applicable -->

### Related issues

<!-- Link related issues here using #issue-number -->

### Checklist

- [x] I have performed a self-review of my code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have updated the documentation accordingly
- [x] My changes generate no new warnings

### Additional notes

<!-- Include any additional information, assumptions, or context that
reviewers might need to understand this PR. -->
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