Skip to content
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

Fix: Handle cases where search header path does not end with "Headers" #431

Merged

Conversation

Ethan-IS
Copy link
Contributor

Description

  • Fixed issue with search header paths not ending with "Headers".

  • Replaced incorrect usage of map with flatMap to resolve mapping error.

  • ${PODS_CONFIGURATION_BUILD_DIR}/ReactCommon/ReactCommon.framework/Headers/react/nativemodule/core not changed to rugby

References

https://github.com/facebook/react-native/blob/7278ff01d7a48607f67079ca4a4e435e3b7125bf/packages/react-native/React/CoreModules/React-CoreModules.podspec#L75

Checklist (I have ...)

  • 🧐 Followed the code style of the rest of the project
  • 📖 Updated the documentation, if necessary
  • 👨🏻‍🔧 Added at least one test which validates that my change is working, if appropriate
  • 👮🏻‍♂️ Run make lint and fixed all warnings
  • ✅ Run make test and fixed all tests

❤️ Thanks for contributing to the 🏈 Rugby!

- Fixed issue with search header paths not ending with "Headers".
- Replaced incorrect usage of `map` with `flatMap` to resolve mapping error.
@swiftyfinch swiftyfinch added the 🧪 Need testing The pull request should be tested manually label Jan 17, 2025
@swiftyfinch swiftyfinch self-requested a review January 17, 2025 18:07
Copy link
Owner

@swiftyfinch swiftyfinch left a comment

Choose a reason for hiding this comment

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

Great job on fixing the issue with the header search paths!
Everything looks good to me. Thank you for your contribution!

@swiftyfinch swiftyfinch merged commit 0f6e963 into swiftyfinch:main Jan 17, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧪 Need testing The pull request should be tested manually
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants