[ci skip] couple more fixes for build pr jar label (#8599)
This commit is contained in:
parent
54daefa0ca
commit
41ab602f47
|
@ -53,16 +53,16 @@ jobs:
|
||||||
if (pull) {
|
if (pull) {
|
||||||
result["pr"] = pull.number;
|
result["pr"] = pull.number;
|
||||||
result["action"] = "paperclip";
|
result["action"] = "paperclip";
|
||||||
core.info(`This is a push action but to a branch with an open PR with the build paperclip label (${JSON.stringify(result)})`);
|
core.notice(`This is a push action but to a branch with an open PR with the build paperclip label (${JSON.stringify(result)})`);
|
||||||
return result;
|
return result;
|
||||||
}
|
}
|
||||||
} else if (event_name === "pull_request" && event.pull_request.labels.find((l) => l.name === "build-pr-jar")) {
|
} else if (event_name === "pull_request" && event.pull_request.labels.find((l) => l.name === "build-pr-jar")) {
|
||||||
result["pr"] = event.pull_request.number;
|
result["pr"] = event.pull_request.number;
|
||||||
result["action"] = "paperclip";
|
result["action"] = "paperclip";
|
||||||
core.info(`This is a pull request action with a build paperclip label (${JSON.stringify(result)})`);
|
core.notice(`This is a pull request action with a build paperclip label (${JSON.stringify(result)})`);
|
||||||
return result;
|
return result;
|
||||||
}
|
}
|
||||||
core.info("This will not build a paperclip jar");
|
core.notice("This will not build a paperclip jar");
|
||||||
return result;
|
return result;
|
||||||
|
|
||||||
- name: Apply Patches
|
- name: Apply Patches
|
||||||
|
@ -72,7 +72,6 @@ jobs:
|
||||||
./gradlew applyPatches --stacktrace
|
./gradlew applyPatches --stacktrace
|
||||||
|
|
||||||
- name: Build
|
- name: Build
|
||||||
if: fromJSON(steps.determine.outputs.result).action == 'build'
|
|
||||||
run: ./gradlew build --stacktrace
|
run: ./gradlew build --stacktrace
|
||||||
|
|
||||||
- name: Create Paperclip Jar
|
- name: Create Paperclip Jar
|
||||||
|
|
|
@ -26,16 +26,12 @@ jobs:
|
||||||
async function updatePR(owner, repo, issue_number, purpose, body) {
|
async function updatePR(owner, repo, issue_number, purpose, body) {
|
||||||
const { data } = await github.rest.issues.get({ owner, repo, issue_number });
|
const { data } = await github.rest.issues.get({ owner, repo, issue_number });
|
||||||
core.debug(JSON.stringify(data, null, 2));
|
core.debug(JSON.stringify(data, null, 2));
|
||||||
|
|
||||||
const marker = `<!-- bot: ${purpose} -->`;
|
const marker = `<!-- bot: ${purpose} -->`;
|
||||||
|
|
||||||
let new_body = data.body ? data.body.trim().split(marker)[0] : "";
|
let new_body = data.body ? data.body.trim().split(marker)[0].trim() : "";
|
||||||
if (!new_body.trim()) {
|
new_body += `\n${marker}\n---\n${body}`;
|
||||||
new_body += `${marker}\n${body}`
|
|
||||||
} else {
|
|
||||||
new_body += `${marker}\n---\n${body}`
|
|
||||||
}
|
|
||||||
|
|
||||||
core.info(`Updating the text body of PR #${issue_number} in ${owner}/${repo}`);
|
core.info(`Updating the text body of PR #${issue_number} in ${owner}/${repo}`);
|
||||||
await github.rest.issues.update({ owner, repo, issue_number, body: new_body });
|
await github.rest.issues.update({ owner, repo, issue_number, body: new_body });
|
||||||
}
|
}
|
||||||
|
@ -43,7 +39,7 @@ jobs:
|
||||||
const { owner, repo } = context.repo;
|
const { owner, repo } = context.repo;
|
||||||
const run_id = ${{ github.event.workflow_run.id }};
|
const run_id = ${{ github.event.workflow_run.id }};
|
||||||
const repo_id = ${{ github.event.repository.id }};
|
const repo_id = ${{ github.event.repository.id }};
|
||||||
|
|
||||||
let pulls = [];
|
let pulls = [];
|
||||||
const event_type = "${{ github.event.workflow_run.event}}";
|
const event_type = "${{ github.event.workflow_run.event}}";
|
||||||
if (event_type === "push") { // if push, it's from the same repo which means `pull_requests` is populated
|
if (event_type === "push") { // if push, it's from the same repo which means `pull_requests` is populated
|
||||||
|
@ -73,7 +69,7 @@ jobs:
|
||||||
if (!artifact) {
|
if (!artifact) {
|
||||||
return core.info("Skipping comment to no matching artifact found");
|
return core.info("Skipping comment to no matching artifact found");
|
||||||
}
|
}
|
||||||
|
|
||||||
const link = `https://nightly.link/${owner}/${repo}/actions/artifacts/${artifact.id}.zip`;
|
const link = `https://nightly.link/${owner}/${repo}/actions/artifacts/${artifact.id}.zip`;
|
||||||
const body = `Download the paperclip jar for this pull request: [${artifact.name}.zip](${link})`;
|
const body = `Download the paperclip jar for this pull request: [${artifact.name}.zip](${link})`;
|
||||||
core.info(`Adding a link to ${link}`);
|
core.info(`Adding a link to ${link}`);
|
||||||
|
|
Loading…
Reference in New Issue