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

Use the right link for docs comment in PR #59

Merged
merged 1 commit into from
Apr 24, 2024
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 25 additions & 13 deletions .github/workflows/comment-docs-download.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ jobs:
if: github.event.workflow_run.event == 'pull_request' && github.event.workflow_run.conclusion == 'success'
runs-on: ubuntu-latest
steps:
- uses: actions/github-script@v6
- uses: actions/github-script@v7
with:
script: |
async function insertUpdateComment(owner, repo, issue_number, purpose, body) {
Expand All @@ -35,11 +35,20 @@ jobs:

const {owner, repo} = context.repo;
const run_id = ${{github.event.workflow_run.id}};
const pull_requests = ${{ toJSON(github.event.workflow_run.pull_requests) }};
if (!pull_requests.length) {
return core.error("This workflow doesn't match any pull requests!");

// Find the PR with the right sha, see https://github.com/orgs/community/discussions/25220
const response = await github.rest.search.issuesAndPullRequests({
q: 'repo:${{ github.repository }} is:pr sha:${{ github.event.workflow_run.head_sha }}',
per_page: 1,
})
const items = response.data.items
if (items.length < 1) {
return core.error("No matching pull requests found");
}
const pullRequestNumber = items[0].number
console.info("Pull request number is", pullRequestNumber)

// get the list of artifacts
const artifacts = await github.paginate(
github.rest.actions.listWorkflowRunArtifacts, {owner, repo, run_id}
);
Expand All @@ -48,15 +57,18 @@ jobs:
return core.error(`No artifacts found`);
}

if (artifacts.length !== 1) {
return core.error(`more than one artifact found`);
}
const link = `https://nightly.link/${owner}/${repo}/actions/artifacts/${artifacts[0].id}.zip`
for (const artifact of artifacts) {
if (artifact.name == "documentation") {
const link = `https://nightly.link/${owner}/${repo}/actions/artifacts/${artifact.id}.zip`

let body = `Here is the build version of the cookbook in this pull request: [documentation.zip](${link}), `;
body += 'you can view it locally by unzipping `documentation.zip` and open the `index.html` with your favorite browser.';

let body = `Here is the build version of the cookbook in this pull request: [documentation.zip](${link}), `;
body += 'you can view it locally by unzipping `documentation.zip` and open the `index.html` with your favorite browser.';
core.info("Review thread message body:", body);
await insertUpdateComment(owner, repo, pullRequestNumber, "link-to-documentation", body);

core.info("Review thread message body:", body);
for (const pr of pull_requests) {
await insertUpdateComment(owner, repo, pr.number, "link-to-wheels", body);
return;
}
}

return core.error(`failed to find "documentation" artifact`);