Skip to content

Conversation

@BrandonStalnaker
Copy link
Collaborator

Summary

  • Added .github/workflows/build-secondary-platforms.yml to build RN with the core sdk associated with each PR.
  • Customized RNExample/ios/Podfile to use the PRs associated core sdk as a development pod
  • RNExample/ios/RNExample/AppDelegate.mm updated to call into the sdk to initialize

Testing Plan

  • here goes (github action)

Reference Issue (For mParticle employees only. Ignore if you are an outside contributor)

@BrandonStalnaker BrandonStalnaker self-assigned this Nov 19, 2024
on: [pull_request]

jobs:
# pod-lint:
Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove all the commented out stuff in this file

.gitignore Outdated
*.stackdump

# ReactCommon subdir shouldn't have Xcode project
/packages/react-native/ReactCommon/**/*.xcodeproj
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we just gitignore the entire /packages directory? I assume these packages and their exact directories may change in the future. Also is there any need to include other things from /packages ever?

@@ -0,0 +1,10 @@
# Add project specific ProGuard rules here.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it possible to remove the Android part of this example project entirely? I know by default it creates both, but it should be possible to make an iOS only RN project.

@BrandonStalnaker BrandonStalnaker merged commit a8bc8a8 into main Nov 27, 2024
11 of 12 checks passed
@BrandonStalnaker BrandonStalnaker deleted the ci/6928-Add-CI-Build-of-RN branch November 27, 2024 15:58
@mparticle-automation
Copy link
Contributor

🎉 This PR is included in version 8.27.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants