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

build: update github workflows #35638

Closed
wants to merge 2 commits into from
Closed

Conversation

targos
Copy link
Member

@targos targos commented Oct 14, 2020

See https://github.com/nodejs/node/actions/runs/306130006 for an example with the set-env deprecation message.

commit 8a7ee81

build: use GITHUB_ENV file to set env variables

The other way is deprecated.

commit b3e6972

build: do not install jq in workflows

It is already installed in the GitHub runners.

It is already installed in the GitHub runners.
@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Oct 14, 2020
@targos
Copy link
Member Author

targos commented Oct 15, 2020

Can we fast track?

@targos targos added fast-track PRs that do not need to wait for 48 hours to land. commit-queue Add this label to land a pull request using GitHub Actions. labels Oct 15, 2020
@github-actions github-actions bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Oct 16, 2020
@github-actions
Copy link
Contributor

Commit Queue failed
- Loading data for nodejs/node/pull/35638
✔  Done loading data for nodejs/node/pull/35638
----------------------------------- PR info ------------------------------------
Title      build: update github workflows (#35638)
Author     Michaël Zasso  (@targos)
Branch     targos:update-workflows -> nodejs:master
Labels     fast-track, meta
Commits    2
 - build: do not install jq in workflows
 - build: use GITHUB_ENV file to set env variables
Committers 1
 - Michaël Zasso 
PR-URL: https://github.com/nodejs/node/pull/35638
Reviewed-By: Richard Lau 
Reviewed-By: Rich Trott 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/35638
Reviewed-By: Richard Lau 
Reviewed-By: Rich Trott 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
--------------------------------------------------------------------------------
   ✔  Last GitHub Actions successful
   ✖  No Jenkins CI runs detected
   ℹ  This PR was created on Wed, 14 Oct 2020 09:19:49 GMT
   ✔  Approvals: 4
   ✔  - Richard Lau (@richardlau): https://github.com/nodejs/node/pull/35638#pullrequestreview-508157108
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/35638#pullrequestreview-508295221
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/35638#pullrequestreview-508666176
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/35638#pullrequestreview-508802687
   ℹ  This PR is being fast-tracked
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu

Commit Queue action: https://github.com/nodejs/node/actions/runs/310055827

targos added a commit that referenced this pull request Oct 16, 2020
It is already installed in the GitHub runners.

PR-URL: #35638
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
targos added a commit that referenced this pull request Oct 16, 2020
The other way is deprecated.

PR-URL: #35638
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@targos
Copy link
Member Author

targos commented Oct 16, 2020

Landed in c143266...cfbbeea

@targos targos closed this Oct 16, 2020
@targos targos deleted the update-workflows branch October 16, 2020 06:10
BethGriggs pushed a commit that referenced this pull request Dec 8, 2020
It is already installed in the GitHub runners.

PR-URL: #35638
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
BethGriggs pushed a commit that referenced this pull request Dec 8, 2020
The other way is deprecated.

PR-URL: #35638
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@BethGriggs BethGriggs mentioned this pull request Dec 10, 2020
@targos targos removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Sep 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fast-track PRs that do not need to wait for 48 hours to land. meta Issues and PRs related to the general management of the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants