Skip to content

Conversation

wcandillon
Copy link
Owner

fixes #246

@wcandillon wcandillon requested a review from Copilot September 18, 2025 07:38
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Updates the README documentation to provide clearer instructions for Three.js integration and iOS setup. The changes focus on improving developer experience by making polyfill requirements more explicit and adding helpful reference links.

  • Restructured Three.js section with dedicated heading and inline code example
  • Replaced polyfill reference link with actual code snippet showing required imports
  • Added Apple documentation reference for Metal validation setup

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.


To run the React Native WebGPU project on the iOS simulator, you need to disable the Metal validation API.
In "Edit Scheme," uncheck "Metal Validation."
In "Edit Scheme," uncheck "Metal Validation.". Learn more [here](https://developer.apple.com/documentation/xcode/validating-your-apps-metal-api-usage/).
Copy link

Copilot AI Sep 18, 2025

Choose a reason for hiding this comment

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

There's an extra period after 'Metal Validation.' - it should be either 'Metal Validation.' or 'Metal Validation'. Learn more...' but not both punctuation marks together.

Suggested change
In "Edit Scheme," uncheck "Metal Validation.". Learn more [here](https://developer.apple.com/documentation/xcode/validating-your-apps-metal-api-usage/).
In "Edit Scheme," uncheck "Metal Validation". Learn more [here](https://developer.apple.com/documentation/xcode/validating-your-apps-metal-api-usage/).

Copilot uses AI. Check for mistakes.

@wcandillon wcandillon merged commit d0e4dbe into main Sep 18, 2025
5 of 6 checks passed
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.

Update the readme

1 participant