opensource/RSSHub
1
0
Fork 0
mirror of https://github.com/DIYgod/RSSHub.git synced 2025-12-03 10:38:03 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: ci wait for label creation before exit (#6655)

Browse Source 
* fix: style and token used on exit

* [CodeFactor] Apply fixes

[ci skip] [skip ci]

Co-authored-by: codefactor-io <support@codefactor.io>
This commit is contained in:
NeverBehave
2021-01-10 19:45:53 -05:00
committed by GitHub
parent b09df0d3e8
commit cb1506e535
3 changed files with 44 additions and 62 deletions
Download Patch File Download Diff File Expand all files Collapse all files

59
scripts/workflow/test-route/identify.js
View File

@@ -1,58 +1,49 @@
const noFound = 'Auto: Route No Found';
module.exports = ({ github, context, core }, body, number) => {
module.exports = async ({github, context, core}, body, number) => {
core.debug(`body: ${body}`);
const m = body.match(/```routes\r\n((.|\r\n)*)```/);
core.debug(`match: ${m}`);
let res = null;
const removeLabel = () => {
github.issues
.removeLabel({
const removeLabel = async () => github.issues.removeLabel({
issue_number: number,
owner: context.repo.owner,
repo: context.repo.repo,
name: noFound
}).catch((e) => { core.warning(e); });
if (m && m[1]) {
res = m[1].trim().split("\r\n");
core.info(`routes detected: ${res}`);
if (res.length > 0 && res[0] === "NOROUTE") {
core.info("PR stated no route, passing");
await removeLabel();
await github.issues.addLabels({
issue_number: number,
owner: context.repo.owner,
repo: context.repo.repo,
name: noFound,
})
.catch((e) => {
core.warning(e);
});
};
if (m && m[1]) {
res = m[1].trim().split('\r\n');
core.info(`routes detected: ${res}`);
if (res.length > 0 && res[0] === 'NOROUTE') {
core.info('PR stated no route, passing');
removeLabel();
github.issues
.addLabels({
issue_number: number,
owner: context.repo.owner,
repo: context.repo.repo,
labels: ['Auto: No Route Needed'],
})
.catch((e) => {
core.warning(e);
});
labels: ['Auto: No Route Needed']
}).catch((e) => { core.warning(e); });
return;
} else if (res.length > 0) {
core.exportVariable('TEST_CONTINUE', true);
removeLabel();
await removeLabel();
return res;
}
}
core.info('seems no route found, failing');
core.info("seems no route found, failing");
github.issues.addLabels({
await github.issues.addLabels({
issue_number: number,
owner: context.repo.owner,
repo: context.repo.repo,
labels: [noFound],
});
labels: [noFound]
}).catch((e) => { core.warning(e); });
throw 'Please follow the PR rules: failed to detect route';
throw "Please follow the PR rules: failed to detect route";
};

42
scripts/workflow/test-route/test.js
View File

@@ -1,6 +1,6 @@
/* eslint-disable */
module.exports = async ({ github, context }, baseUrl, routes, number) => {
module.exports = async ({github, context}, baseUrl, routes, number) => {
if (routes[0] === 'NOROUTE') {
return;
}
@@ -13,10 +13,10 @@ module.exports = async ({ github, context }, baseUrl, routes, number) => {
let com = 'Successfully generated as following:\n\n';
for (const lks of links) {
console.log('testing route: ', lks);
console.log("testing route: ", lks)
// Intended, one at a time
const res = await github.request(`GET ${lks}`).catch((err) => {
com += `
const res = await github.request(`GET ${lks}`).catch(err => {
com+= `
<details>
<summary><a href="${lks}">${lks}</a> - **Failed**</summary>
@@ -42,24 +42,16 @@ module.exports = async ({ github, context }, baseUrl, routes, number) => {
`;
}
}
github.issues
.addLabels({
issue_number: number,
owner: context.repo.owner,
repo: context.repo.repo,
labels: ['Auto: Route Test Complete'],
})
.catch((e) => {
core.warning(e);
});
github.issues
.createComment({
issue_number: number,
owner: context.repo.owner,
repo: context.repo.repo,
body: com,
})
.catch((e) => {
core.warning(e);
});
};
github.issues.addLabels({
issue_number: number,
owner: context.repo.owner,
repo: context.repo.repo,
labels: ['Auto: Route Test Complete']
}).catch((e) => { core.warning(e) })
github.issues.createComment({
issue_number: number,
owner: context.repo.owner,
repo: context.repo.repo,
body: com
}).catch((e) => { core.warning(e) })
}