FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

tools: make GH Actions workflows work if default branch is not master by aduh95 · Pull Request #38516 · nodejs/node · GitHub

/ node Public

tools: make GH Actions workflows work if default branch is not master - #38516

Closed
aduh95 wants to merge 1 commit into
nodejs:masterfrom
aduh95:workflow-main
Closed

tools: make GH Actions workflows work if default branch is not master#38516
aduh95 wants to merge 1 commit into
nodejs:masterfrom
aduh95:workflow-main

Conversation

aduh95 commented May 3, 2021

Copy link
Copy Markdown
Contributor

The workflows should be able to work whether the default branch is named master or main. This should ease the transition if/when we switch to main in the future.

github-actions Bot added the meta Issues and PRs related to the general management of the project. label May 3, 2021
aduh95 added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label May 3, 2021
jasnell added the fast-track PRs that do not need to wait for 48 hours to land. label May 3, 2021

github-actions Bot commented May 3, 2021

Copy link
Copy Markdown
Contributor

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

jasnell commented May 4, 2021

Copy link
Copy Markdown
Member

Landed in b034151

jasnell closed this May 4, 2021
jasnell pushed a commit that referenced this pull request May 4, 2021
PR-URL: #38516
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>
targos pushed a commit that referenced this pull request May 17, 2021
PR-URL: #38516
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>
targos pushed a commit that referenced this pull request May 30, 2021
PR-URL: #38516
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>
targos pushed a commit that referenced this pull request Jun 5, 2021
PR-URL: #38516
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>
targos pushed a commit that referenced this pull request Jun 5, 2021
PR-URL: #38516
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>
targos pushed a commit that referenced this pull request Jun 11, 2021
PR-URL: #38516
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no outstanding review comments, and a CI started. 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


Back | FazBrowse Home | New Git URL