mirror of
https://github.com/kremalicious/metamask-extension.git
synced 2024-11-22 18:00:18 +01:00
Remove unused GitHub Action workflow (#19660)
This GitHub action workflow was disabled, but was still running setup steps. It has now been removed entirely. We can re-introduce it again later once the problem that led to it being disabled has been fixed. The associated npm script and JavaScript module have been removed as well.
This commit is contained in:
parent
e7e55dd873
commit
55a8ff249c
177
.github/scripts/label-prs.ts
vendored
177
.github/scripts/label-prs.ts
vendored
@ -1,177 +0,0 @@
|
|||||||
import * as core from '@actions/core';
|
|
||||||
import { context, getOctokit } from '@actions/github';
|
|
||||||
import { GitHub } from '@actions/github/lib/utils';
|
|
||||||
|
|
||||||
main().catch((error: Error): void => {
|
|
||||||
console.error(error);
|
|
||||||
process.exit(1);
|
|
||||||
});
|
|
||||||
|
|
||||||
async function main(): Promise<void> {
|
|
||||||
const token = process.env.GITHUB_TOKEN;
|
|
||||||
|
|
||||||
if (!token) {
|
|
||||||
core.setFailed('GITHUB_TOKEN not found');
|
|
||||||
process.exit(1);
|
|
||||||
}
|
|
||||||
|
|
||||||
const octokit = getOctokit(token);
|
|
||||||
|
|
||||||
const headRef = context.payload.pull_request?.head.ref || '';
|
|
||||||
|
|
||||||
let issueNumber = await getIssueNumberFromPullRequestBody();
|
|
||||||
if (issueNumber === "") {
|
|
||||||
bailIfIsBranchNameInvalid(headRef);
|
|
||||||
bailIfIsNotFeatureBranch(headRef);
|
|
||||||
issueNumber = getIssueNumberFromBranchName(headRef);
|
|
||||||
}
|
|
||||||
|
|
||||||
if (Number(issueNumber) === 0) {
|
|
||||||
process.exit(0);
|
|
||||||
}
|
|
||||||
|
|
||||||
await updateLabels(octokit, issueNumber);
|
|
||||||
}
|
|
||||||
|
|
||||||
async function getIssueNumberFromPullRequestBody(): Promise<string> {
|
|
||||||
console.log("Checking if the PR's body references an issue...");
|
|
||||||
|
|
||||||
let ISSUE_LINK_IN_PR_DESCRIPTION_REGEX =
|
|
||||||
/(close|closes|closed|fix|fixes|fixed|resolve|resolves|resolved)\s#\d+/gi;
|
|
||||||
|
|
||||||
const prBody = await getPullRequestBody();
|
|
||||||
|
|
||||||
let matches = prBody.match(ISSUE_LINK_IN_PR_DESCRIPTION_REGEX);
|
|
||||||
if (!matches || matches?.length === 0) {
|
|
||||||
console.log(
|
|
||||||
'No direct link can be drawn between the PR and an issue from the PR body because no issue number was referenced.',
|
|
||||||
);
|
|
||||||
return "";
|
|
||||||
}
|
|
||||||
|
|
||||||
if (matches?.length > 1) {
|
|
||||||
console.log(
|
|
||||||
'No direct link can be drawn between the PR and an issue from the PR body because more than one issue number was referenced.',
|
|
||||||
);
|
|
||||||
return "";
|
|
||||||
}
|
|
||||||
|
|
||||||
const ISSUE_NUMBER_REGEX = /\d+/;
|
|
||||||
const issueNumber = matches[0].match(ISSUE_NUMBER_REGEX)?.[0] || '';
|
|
||||||
|
|
||||||
console.log(`Found issue number ${issueNumber} in PR body.`);
|
|
||||||
|
|
||||||
return issueNumber;
|
|
||||||
}
|
|
||||||
|
|
||||||
async function getPullRequestBody(): Promise<string> {
|
|
||||||
if (context.eventName !== 'pull_request') {
|
|
||||||
console.log('This action should only run on pull_request events.');
|
|
||||||
process.exit(1);
|
|
||||||
}
|
|
||||||
|
|
||||||
const prBody = context.payload.pull_request?.body || '';
|
|
||||||
return prBody;
|
|
||||||
}
|
|
||||||
|
|
||||||
function bailIfIsBranchNameInvalid(branchName: string): void {
|
|
||||||
const BRANCH_REGEX =
|
|
||||||
/^(main|develop|(ci|chore|docs|feat|feature|fix|perf|refactor|revert|style)\/\d*(?:[-](?![-])\w*)*|Version-v\d+\.\d+\.\d+)$/;
|
|
||||||
const isValidBranchName = new RegExp(BRANCH_REGEX).test(branchName);
|
|
||||||
|
|
||||||
if (!isValidBranchName) {
|
|
||||||
console.log('This branch name does not follow the convention.');
|
|
||||||
console.log(
|
|
||||||
'Here are some example branch names that are accepted: "fix/123-description", "feat/123-longer-description", "feature/123", "main", "develop", "Version-v10.24.2".',
|
|
||||||
);
|
|
||||||
console.log(
|
|
||||||
'No issue could be linked to this PR, so no labels were copied',
|
|
||||||
);
|
|
||||||
|
|
||||||
process.exit(0);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
function bailIfIsNotFeatureBranch(branchName: string): void {
|
|
||||||
if (
|
|
||||||
branchName === 'main' ||
|
|
||||||
branchName === 'develop' ||
|
|
||||||
branchName.startsWith('Version-v')
|
|
||||||
) {
|
|
||||||
console.log(`${branchName} is not a feature branch.`);
|
|
||||||
console.log(
|
|
||||||
'No issue could be linked to this PR, so no labels were copied',
|
|
||||||
);
|
|
||||||
process.exit(0);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
async function updateLabels(octokit: InstanceType<typeof GitHub>, issueNumber: string): Promise<void> {
|
|
||||||
interface ILabel {
|
|
||||||
name: string;
|
|
||||||
};
|
|
||||||
|
|
||||||
const owner = context.repo.owner;
|
|
||||||
const repo = context.repo.repo;
|
|
||||||
|
|
||||||
const issue = await octokit.rest.issues.get({
|
|
||||||
owner: owner,
|
|
||||||
repo: repo,
|
|
||||||
issue_number: Number(issueNumber),
|
|
||||||
});
|
|
||||||
|
|
||||||
const getNameFromLabel = (label: ILabel): string => label.name
|
|
||||||
|
|
||||||
const issueLabels = issue.data.labels.map(label => getNameFromLabel(label as ILabel));
|
|
||||||
|
|
||||||
const prNumber = context.payload.number;
|
|
||||||
|
|
||||||
const pr = await octokit.rest.issues.get({
|
|
||||||
owner: owner,
|
|
||||||
repo: repo,
|
|
||||||
issue_number: prNumber,
|
|
||||||
});
|
|
||||||
|
|
||||||
const startingPRLabels = pr.data.labels.map(label => getNameFromLabel(label as ILabel));
|
|
||||||
|
|
||||||
const dedupedFinalPRLabels = [
|
|
||||||
...new Set([...startingPRLabels, ...issueLabels]),
|
|
||||||
];
|
|
||||||
|
|
||||||
const hasIssueAdditionalLabels = !sortedArrayEqual(
|
|
||||||
startingPRLabels,
|
|
||||||
dedupedFinalPRLabels,
|
|
||||||
);
|
|
||||||
if (hasIssueAdditionalLabels) {
|
|
||||||
await octokit.rest.issues.update({
|
|
||||||
owner,
|
|
||||||
repo,
|
|
||||||
issue_number: prNumber,
|
|
||||||
labels: dedupedFinalPRLabels,
|
|
||||||
});
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
function getIssueNumberFromBranchName(branchName: string): string {
|
|
||||||
console.log('Checking if the branch name references an issue...');
|
|
||||||
|
|
||||||
let issueNumber: string;
|
|
||||||
if (branchName.split('/').length > 1) {
|
|
||||||
issueNumber = branchName.split('/')[1].split('-')[0];
|
|
||||||
} else {
|
|
||||||
issueNumber = branchName.split('-')[0];
|
|
||||||
}
|
|
||||||
|
|
||||||
console.log(`Found issue number ${issueNumber} in branch name.`);
|
|
||||||
|
|
||||||
return issueNumber;
|
|
||||||
}
|
|
||||||
|
|
||||||
function sortedArrayEqual(array1: string[], array2: string[]): boolean {
|
|
||||||
const lengthsAreEqual = array1.length === array2.length;
|
|
||||||
const everyElementMatchesByIndex = array1.every(
|
|
||||||
(value: string, index: number): boolean => value === array2[index],
|
|
||||||
);
|
|
||||||
|
|
||||||
return lengthsAreEqual && everyElementMatchesByIndex;
|
|
||||||
}
|
|
27
.github/workflows/label-prs.yml
vendored
27
.github/workflows/label-prs.yml
vendored
@ -1,27 +0,0 @@
|
|||||||
name: Label PR
|
|
||||||
|
|
||||||
on:
|
|
||||||
pull_request:
|
|
||||||
types: [assigned, opened, edited, synchronize, reopened]
|
|
||||||
|
|
||||||
jobs:
|
|
||||||
label-pr:
|
|
||||||
runs-on: ubuntu-latest
|
|
||||||
permissions:
|
|
||||||
issues: write
|
|
||||||
pull-requests: write
|
|
||||||
|
|
||||||
steps:
|
|
||||||
- name: Checkout code
|
|
||||||
uses: actions/checkout@v3
|
|
||||||
|
|
||||||
- name: Use Node.js
|
|
||||||
uses: actions/setup-node@v3
|
|
||||||
with:
|
|
||||||
node-version: '16'
|
|
||||||
|
|
||||||
- name: Install Yarn
|
|
||||||
run: npm install -g yarn
|
|
||||||
|
|
||||||
- name: Install dependencies
|
|
||||||
run: yarn
|
|
@ -95,7 +95,6 @@
|
|||||||
"fitness-functions": "ts-node development/fitness-functions/index.ts",
|
"fitness-functions": "ts-node development/fitness-functions/index.ts",
|
||||||
"generate-beta-commit": "node ./development/generate-beta-commit.js",
|
"generate-beta-commit": "node ./development/generate-beta-commit.js",
|
||||||
"validate-branch-name": "validate-branch-name",
|
"validate-branch-name": "validate-branch-name",
|
||||||
"label-prs": "ts-node ./.github/scripts/label-prs.ts",
|
|
||||||
"add-release-label-to-pr-and-linked-issues": "ts-node ./.github/scripts/add-release-label-to-pr-and-linked-issues.ts"
|
"add-release-label-to-pr-and-linked-issues": "ts-node ./.github/scripts/add-release-label-to-pr-and-linked-issues.ts"
|
||||||
},
|
},
|
||||||
"resolutions": {
|
"resolutions": {
|
||||||
|
Loading…
Reference in New Issue
Block a user