forked from facebook/react-native
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add automatic RN version checking workflow (facebook#36075)
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
- Loading branch information
1 parent
dc98ab0
commit 1f6bc04
Showing
5 changed files
with
284 additions
and
188 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,137 @@ | ||
/** | ||
* Copyright (c) Meta Platforms, Inc. and affiliates. | ||
* | ||
* This source code is licensed under the MIT license found in the | ||
* LICENSE file in the root directory of this source tree. | ||
* | ||
* @format | ||
*/ | ||
|
||
module.exports = async (github, context, label) => { | ||
const closeIssue = async () => { | ||
await github.rest.issues.update({ | ||
issue_number: context.issue.number, | ||
owner: context.repo.owner, | ||
repo: context.repo.repo, | ||
state: 'closed', | ||
}); | ||
}; | ||
|
||
const addComment = async comment => { | ||
await github.rest.issues.createComment({ | ||
issue_number: context.issue.number, | ||
owner: context.repo.owner, | ||
repo: context.repo.repo, | ||
body: comment, | ||
}); | ||
}; | ||
|
||
const requestAuthorFeedback = async () => { | ||
// Remove the triage label if it exists (ignore the 404 if not; it's not expected to always be there) | ||
try { | ||
await github.rest.issues.removeLabel({ | ||
issue_number: context.issue.number, | ||
owner: context.repo.owner, | ||
repo: context.repo.repo, | ||
name: 'Needs: Triage :mag:', | ||
}); | ||
} catch {} | ||
|
||
await github.rest.issues.addLabels({ | ||
issue_number: context.issue.number, | ||
owner: context.repo.owner, | ||
repo: context.repo.repo, | ||
labels: ['Needs: Author Feedback'], | ||
}); | ||
}; | ||
|
||
switch (label) { | ||
case 'Type: Invalid': | ||
await addComment( | ||
`| :warning: | Issue is Invalid |\n` + | ||
`| --- | --- |\n` + | ||
`| :information_source: | This issue doesn't match any of the expected types for this repository - closing. |`, | ||
); | ||
await closeIssue(); | ||
return; | ||
case 'Type: Question': | ||
await addComment( | ||
`| :warning: | Issue is a Question |\n` + | ||
`| --- | --- |\n` + | ||
`| :information_source: | We are using GitHub issues exclusively to track bugs in React Native. GitHub may not be the ideal place to ask a question, but you can try asking over on [Stack Overflow](http://stackoverflow.com/questions/tagged/react-native), or on [Reactiflux](https://www.reactiflux.com/). |`, | ||
); | ||
await closeIssue(); | ||
return; | ||
case 'Type: Docs': | ||
await addComment( | ||
`| :warning: | Documentation Issue |\n` + | ||
`| --- | --- |\n` + | ||
`| :information_source: | Please report documentation issues in the [react-native-website](https://github.com/facebook/react-native-website/issues) repository. |`, | ||
); | ||
await closeIssue(); | ||
return; | ||
case 'Resolution: For Stack Overflow': | ||
await addComment( | ||
`| :warning: | Issue is a Question |\n` + | ||
`| --- | --- |\n` + | ||
`| :information_source: | We are using GitHub issues exclusively to track bugs in the core React Native library. Please try asking over on [Stack Overflow](http://stackoverflow.com/questions/tagged/react-native) as it is better suited for this type of question. |`, | ||
); | ||
await closeIssue(); | ||
return; | ||
case 'Type: Expo': | ||
await addComment( | ||
`| :warning: | Issue is Related to Expo |\n` + | ||
`| --- | --- |\n` + | ||
`| :information_source: | It looks like your issue is related to Expo and not React Native core. Please open your issue in [Expo's repository](https://github.com/expo/expo/issues/new). If you are able to create a repro that showcases that this issue is also happening in React Native vanilla, we will be happy to re-open. |`, | ||
); | ||
await closeIssue(); | ||
return; | ||
case 'Needs: Issue Template': | ||
await addComment( | ||
`| :warning: | Missing Required Fields |\n` + | ||
`| --- | --- |\n` + | ||
`| :information_source: | It looks like your issue may be missing some necessary information. GitHub provides an example template whenever a [new issue is created](https://github.com/facebook/react-native/issues/new?template=bug_report.md). Could you go back and make sure to fill out the template? You may edit this issue, or close it and open a new one. |`, | ||
); | ||
await requestAuthorFeedback(); | ||
return; | ||
case 'Needs: Environment Info': | ||
await addComment( | ||
`| :warning: | Missing Environment Information |\n` + | ||
`| --- | --- |\n` + | ||
`| :information_source: | Your issue may be missing information about your development environment. You can obtain the missing information by running <code>react-native info</code> in a console. |`, | ||
); | ||
await requestAuthorFeedback(); | ||
return; | ||
case 'Needs: Verify on Latest Version': | ||
await addComment( | ||
`| :warning: | Newer Version of React Native is Available! |\n` + | ||
`| --- | --- |\n` + | ||
`| :information_source: | You are on a supported minor version, but it looks like there's a newer patch available. Please [upgrade](https://reactnative.dev/docs/upgrading) to the highest patch for your minor or latest and verify if the issue persists (alternatively, create a new project and repro the issue in it). If it does not repro, please let us know so we can close out this issue. This helps us ensure we are looking at issues that still exist in the most recent releases. |`, | ||
); | ||
return; | ||
case 'Needs: Version Info': | ||
await addComment( | ||
`| :warning: | Add or Reformat Version Info |\n` + | ||
`| --- | --- |\n` + | ||
`| :information_source: | We could not find or parse the version number of React Native in your issue report. Please use the template, and report your version including major, minor, and patch numbers - e.g. 0.70.2 |`, | ||
); | ||
await requestAuthorFeedback(); | ||
return; | ||
case 'Needs: Repro': | ||
await addComment( | ||
`| :warning: | Missing Reproducible Example |\n` + | ||
`| --- | --- |\n` + | ||
`| :information_source: | It looks like your issue is missing a reproducible example. Please provide a [Snack](https://snack.expo.dev) or a repository that demonstrates the issue you are reporting in a [minimal, complete, and reproducible](https://stackoverflow.com/help/minimal-reproducible-example) manner. |`, | ||
); | ||
await requestAuthorFeedback(); | ||
return; | ||
case 'Type: Unsupported Version': | ||
await addComment( | ||
`| :warning: | Unsupported Version of React Native |\n` + | ||
`| --- | --- |\n` + | ||
`| :information_source: | It looks like your issue or the example you provided uses an [unsupported version of React Native](https://github.com/reactwg/react-native-releases/blob/main/README.md#releases-support-policy). Due to the number of issues we receive, we're currently only accepting new issues against one of the supported versions. Please [upgrade](https://reactnative.dev/docs/upgrading) to latest and verify if the issue persists (alternatively, create a new project and repro the issue in it). If you cannot upgrade, please open your issue on [StackOverflow](https://stackoverflow.com/questions/tagged/react-native) to get further community support. |`, | ||
); | ||
await requestAuthorFeedback(); | ||
return; | ||
} | ||
}; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.