Skip to content

Conversation

coado
Copy link
Owner

@coado coado commented Jun 27, 2025

Summary:

Changelog:

Test Plan:

coado and others added 7 commits June 23, 2025 01:16
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
Add diff-api-snapshot action to danger
@coado
Copy link
Owner Author

coado commented Jun 27, 2025

Fails
🚫

📋 Verify Changelog Format - See Changelog format


Warnings
⚠️ ❗ 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.

This change was flagged as: BREAKING

Generated by 🚫 dangerJS against 2963227

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