Skip to content

Conversation

@richardlau
Copy link
Member

Use the same commit SHA for this action as used in our other workflows.

Refs: #46169
Refs: #45022 (comment)


The workflow is currently failing: https://github.com/nodejs/node/actions/runs/5215729249/jobs/9413680952

Run gr2m/create-or-update-pull-request-action@df20b2c073090271599a08c55ae26e0c3522b329Error: File not found: '/home/runner/work/_actions/gr2m/create-or-update-pull-request-action/df20b2c073090271599a08c55ae26e0c3522b329/dist/index.js'

Use the same commit SHA for this action as used in our other workflows. Refs: #46169 Refs: #45022 (comment)
@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/actions
  • @nodejs/security-wg

@nodejs-github-botnodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Jun 8, 2023
@richardlau
Copy link
MemberAuthor

Copy link
Member

@marco-ippolitomarco-ippolito left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@richardlau
Copy link
MemberAuthor

Workflow run with this PR: https://github.com/nodejs/node/actions/runs/5216284702/jobs/9414851477

This workflow run opened #48399 which looks correct, except because I ran the workflow from the branch from this PR it included the commit from this PR in #48399. I'll close off #48399 and we can rerun the fixed workflow from the main branch once this PR lands.

@bnoordhuisbnoordhuis added the fast-track PRs that do not need to wait for 48 hours to land. label Jun 9, 2023
@github-actions
Copy link
Contributor

Fast-track has been requested by @bnoordhuis. Please 👍 to approve.

@richardlaurichardlau added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 9, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 9, 2023
@nodejs-github-botnodejs-github-bot merged commit 6a3403c into mainJun 9, 2023
@nodejs-github-bot
Copy link
Collaborator

Landed in 6a3403c

@nodejs-github-botnodejs-github-bot deleted the update-openssl-workflow branch June 9, 2023 11:54
RafaelGSS pushed a commit that referenced this pull request Jul 3, 2023
Use the same commit SHA for this action as used in our other workflows. Refs: #46169 Refs: #45022 (comment) PR-URL: #48398 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
@RafaelGSSRafaelGSS mentioned this pull request Jul 3, 2023
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
Use the same commit SHA for this action as used in our other workflows. Refs: nodejs#46169 Refs: nodejs#45022 (comment) PR-URL: nodejs#48398 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
Use the same commit SHA for this action as used in our other workflows. Refs: nodejs#46169 Refs: nodejs#45022 (comment) PR-URL: nodejs#48398 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
ruyadorno pushed a commit that referenced this pull request Aug 29, 2023
Use the same commit SHA for this action as used in our other workflows. Refs: #46169 Refs: #45022 (comment) PR-URL: #48398 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
@ruyadornoruyadorno mentioned this pull request Aug 29, 2023
ruyadorno pushed a commit that referenced this pull request Sep 1, 2023
Use the same commit SHA for this action as used in our other workflows. Refs: #46169 Refs: #45022 (comment) PR-URL: #48398 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast-trackPRs that do not need to wait for 48 hours to land.metaIssues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@richardlau@nodejs-github-bot@bnoordhuis@tniessen@marco-ippolito@Mesteery