Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add automatic RN version checking workflow #36075

Closed
wants to merge 10 commits into from

Conversation

SlyCaptainFlint
Copy link
Contributor

Summary

Adding automatic RN version checking github workflow, which will verify the version of RN listed on all new issues filed in the repository.
Additionally, this change refactors the existing labeler workflow to make it re-usable by the version check workflow. The change also creates a logical place to add future automatic detection checks, like auto-verification of repro, template, etc.
This is technically not new functionality, as the react-native-bot does this sometimes, but this should be a lot more reliable.
The logic for valid release checking follows what is listed in the documentation - valid versions are current and N-2 minors, with the highest available patches.

Changelog

[INTERNAL] [FIXED] - Made the automated RN version checking workflow more reliable

Test Plan

I have verified a variety of different versions on issues here: https://github.com/SlyCaptainFlint/react-native/issues
I have also re-verified all the tags that were previously handled by the labeler workflow, since I have refactored it. Please take a look at both the open and closed issues in the linked repo for examples.

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Feb 7, 2023
@analysis-bot
Copy link

analysis-bot commented Feb 7, 2023

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,458,541 +0
android hermes armeabi-v7a 7,781,811 +0
android hermes x86 8,934,442 +0
android hermes x86_64 8,791,524 +0
android jsc arm64-v8a 9,092,729 +0
android jsc armeabi-v7a 8,290,826 +0
android jsc x86 9,143,541 +0
android jsc x86_64 9,402,401 +0

Base commit: c5bc3f1
Branch: main

@facebook-github-bot
Copy link
Contributor

@SlyCaptainFlint has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

2 similar comments
@facebook-github-bot
Copy link
Contributor

@SlyCaptainFlint has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@SlyCaptainFlint has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Copy link
Contributor

@kelset kelset left a comment

Choose a reason for hiding this comment

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

left a few comments, overall I like the refactoring - thanks for working on this!

.github/workflows/actOnLabel.js Show resolved Hide resolved
.github/workflows/actOnLabel.js Show resolved Hide resolved
.github/workflows/actOnLabel.js Outdated Show resolved Hide resolved
.github/workflows/on-issue-labeled.yml Outdated Show resolved Hide resolved
.github/workflows/verifyVersion.js Outdated Show resolved Hide resolved
.github/workflows/verifyVersion.js Outdated Show resolved Hide resolved
Copy link
Contributor

@kelset kelset left a comment

Choose a reason for hiding this comment

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

I think this is a great improvement - we'll just need to keep an eye on the situation to see how the community reacts to the changes.

@facebook-github-bot
Copy link
Contributor

@SlyCaptainFlint has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@SlyCaptainFlint
Copy link
Contributor Author

/rebase

@facebook-github-bot
Copy link
Contributor

@SlyCaptainFlint has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@SlyCaptainFlint has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@SlyCaptainFlint merged this pull request in 26b2bb5.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Feb 23, 2023
OlimpiaZurek pushed a commit to OlimpiaZurek/react-native that referenced this pull request May 22, 2023
Summary:
Adding automatic RN version checking github workflow, which will verify the version of RN listed on all new issues filed in the repository.
Additionally, this change refactors the existing labeler workflow to make it re-usable by the version check workflow. The change also creates a logical place to add future automatic detection checks, like auto-verification of repro, template, etc.
This is technically not new functionality, as the react-native-bot does this _sometimes_, but this should be a lot more reliable.
The logic for valid release checking follows what is listed in the documentation - valid versions are current and N-2 minors, with the highest available patches.

## Changelog

[INTERNAL] [FIXED] - Made the automated RN version checking workflow more reliable

Pull Request resolved: facebook#36075

Test Plan:
I have verified a variety of different versions on issues here: https://github.com/SlyCaptainFlint/react-native/issues
I have also re-verified all the tags that were previously handled by the labeler workflow, since I have refactored it. Please take a look at both the open and closed issues in the linked repo for examples.

Reviewed By: cortinico

Differential Revision: D43089150

Pulled By: SlyCaptainFlint

fbshipit-source-id: 7da67f5cb2a4875f22e1f9e46d7ca07d43f3e135
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants