Skip to content

Conversation

fabOnReact
Copy link
Owner

Summary

  • add platform connector examples for WearOS and watchOS
  • document step-by-step setup for WearOS and watchOS
  • add troubleshooting and contribution guides

Testing

  • yarn lint (fails: eslint-plugin-ft-flow missing)
  • yarn test (fails: Cannot find module '@react-native/babel-preset')
  • npx tsc --noEmit

https://chatgpt.com/codex/tasks/task_e_68b7027cbff8832086ce6d4e58abca7d

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

Codex Review: Here are some suggestions.

Reply with @codex fix comments to fix any unresolved comments.

About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".

Comment on lines 24 to +28
"strict": true,
"target": "esnext",
"verbatimModuleSyntax": true
}
},
"exclude": ["node_modules", "examples"]

Choose a reason for hiding this comment

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

[P1] Exclude new examples from TypeScript build

Adding "exclude": ["node_modules", "examples"] here only affects tsc --noEmit, because tsconfig.build.json defines its own exclude array and overrides this change. When yarn prepare (bob build) runs, the TypeScript project referenced in tsconfig.build.json still compiles the new files in examples/, which fail with missing modules and unused variables, causing the build to stop. To avoid blocking yarn prepare/publishing, tsconfig.build.json also needs to exclude the examples folder.

Useful? React with 👍 / 👎.

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.

1 participant