Skip to content

Conversation

@coado
Copy link
Owner

@coado coado commented Jun 16, 2025

Summary:

Changelog:

Test Plan:

@coado
Copy link
Owner Author

coado commented Jun 16, 2025

Fails
🚫 ❗ Javascript API change detected - This PR commits an update to ReactNativeApi.d.ts, indicating a change to React Native's public JavaScript API. Please include a clear changelog message. This change will be subject to extra review.
🚫

📋 Verify Changelog Format - See Changelog format

Generated by 🚫 dangerJS against 458777a

@coado coado closed this Jun 16, 2025
coado added a commit that referenced this pull request Jun 23, 2025
Summary:
This is a work in progress PR that adds breaking change detection to danger.

The danger is ran on a base revision from which there is no easy access to the API snapshot from the PR. The Github action fetches the PR and stores the "current" snapshot in the temporary directory which is then used in the `diff-api-snapshot` script. It is compared to the previous snapshot from the base revision which can be easily accessed by reading from `packages/react-native/ReactNativeApi.d.ts`.


## Changelog:

<!-- Help reviewers and the release process by writing your own changelog entry.

Pick one each for the category and type tags:

[ANDROID|GENERAL|IOS|INTERNAL] [BREAKING|ADDED|CHANGED|DEPRECATED|REMOVED|FIXED|SECURITY] - Message

For more details, see:
https://reactnative.dev/contributing/changelogs-in-pull-requests


Test Plan:
Tested on react-native fork with personal access token. 

 {F1979339175} 

#16

Differential Revision: D76735630

Pulled By: coado
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.

2 participants