Feature/cross platform ios #2
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…port - Introduced a new document outlining the goals, tasks, and timelines for achieving complete cross-platform support for macOS, iOS, and Android. - Included specific tasks for iOS and Android platform integration, as well as a unified testing process. - Simplified the roadmap by focusing on core functionalities and removing complex tasks related to engine optimization.
- Add iOS platform detection and conditional compilation in CMakeLists.txt - Implement iOS-specific DeviceInfo module using Foundation APIs - Add iOS device and simulator build targets to Makefile - Support platform-specific framework linking (IOKit for macOS, UIKit for iOS) - Set appropriate deployment targets (macOS 10.15+, iOS 12.0+) - Create separate build directories for iOS device (build_ios) and simulator (build_ios_sim) - Maintain backward compatibility with existing macOS build system
- Introduced a new executable for testing the DeviceInfo module on iOS. - Added a comprehensive iOS testing script (`test_ios.sh`) to facilitate running various tests on the iOS simulator. - Created documentation (`iOS_TESTING.md`) detailing the testing process and available test types. - Implemented a sample test for the DeviceInfo module to verify functionality and performance metrics.
- Consolidated iOS build configurations by removing separate simulator build targets in the Makefile. - Updated the iOS testing script to reference the unified build directory for iOS. - Enhanced documentation in the Makefile to reflect changes in build and test commands for iOS. - Introduced new test targets for running comprehensive iOS tests directly from the Makefile.
…g into existing integration test - Deleted the standalone iOS DeviceInfo test executable. - Updated the integration test to include performance testing for DeviceInfo methods. - Modified the testing script to reflect changes in test execution and reporting. - Enhanced the integration test output to provide detailed performance metrics.
…te Makefile - Introduced a new executable for lightweight performance testing (`test_performance`) to assess Bridge communication efficiency. - Updated the Makefile to include a target for running the performance test and added relevant documentation for the new test command. - Removed the performance testing function from the integration test to streamline the testing process.
- Added iOS-specific configurations in CMakeLists.txt to include an Objective-C helper file for integration tests. - Implemented custom commands to copy the JavaScript bundle and test scripts into the iOS app package post-build. - Updated the test_integration.cpp file to utilize the new Objective-C functions for retrieving bundle and resource paths on iOS. - Ensured that the integration test can now dynamically load resources based on the platform, improving cross-platform compatibility.
…integration tests - Renamed the Objective-C helper file from `.m` to `.mm` to support C++ interoperability. - Updated CMakeLists.txt to reflect the change in file extension for the iOS integration test executable. - Removed the old Objective-C helper file and implemented the necessary functions in the new Objective-C++ file.
- Introduced a new submodule for the vibe-writing-workflow repository to streamline project management. - Updated claude.md to enhance the development strategy section, including key project paths and build commands. - Added detailed build and testing commands for various platforms in the documentation to improve usability.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.