Skip to content

test: cover BigInt view props on mount and update - #1665

Open
hannojg wants to merge 1 commit into
mainfrom
hannojg/test-bigint-view-props
Open

hannojg wants to merge 1 commit into
mainfrom
hannojg/test-bigint-view-props

Conversation

@hannojg

@hannojg hannojg commented Sep 26, 2026 •

Copy link
Copy Markdown
Member

In this PR:

AI did something that would channel our custom nitro props through RN's folly::dynamic conversion system. This would cause issues & crashes.

This PR adds a BigInt to the props of the test view. This way AI should notice during testing that it can't do that, as this would crash in the folly converter.

@vercel

vercel Bot commented Sep 26, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
nitro-docs Skipped Skipped Sep 26, 2026 10:57pm UTC

Request Review

@hannojg
hannojg requested a review from mrousavy September 26, 2026 23:01
@hannojg
hannojg marked this pull request as ready for review September 26, 2026 23:02

This branch was previously deployed

1 inactive deployment
Preview — 0f003ec4 Deployed Sep 26, 2026 by vercel[bot]
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.

1 participant