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

tools: add release host var to promotion script by ruyadorno · Pull Request #45913 · nodejs/node · GitHub

/ node Public

tools: add release host var to promotion script - #45913

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
ruyadorno:add-host-env-var-to-release-script
Jan 1, 2023
Merged

tools: add release host var to promotion script#45913
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
ruyadorno:add-host-env-var-to-release-script

Conversation

ruyadorno commented Dec 19, 2022
edited
Loading

Copy link
Copy Markdown
Member

Adds a NODEJS_RELEASE_HOST environment variable that enable releasers to provide a custom proxy host to connect to when performing the promotion steps of a given release.

cc @nodejs/releasers

nodejs-github-bot commented Dec 19, 2022
edited by ruyadorno
Loading

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/tsc

nodejs-github-bot added the tools Issues and PRs related to the tools directory. label Dec 19, 2022
Comment thread doc/contributing/releases.md Outdated
ruyadorno force-pushed the add-host-env-var-to-release-script branch 2 times, most recently from 8d30d64 to 4fcdf24 Compare December 19, 2022 21:17
ruyadorno added the request-ci Add this label to start a Jenkins CI on a PR. label Dec 19, 2022
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Dec 19, 2022

This comment was marked as outdated.

nitindudeja2107 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Copy link
Copy Markdown
Collaborator

Comment thread doc/contributing/releases.md Outdated
RafaelGSS added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Dec 22, 2022
Comment thread tools/release.sh Outdated
Adds a `NODEJS_RELEASE_HOST` environment variable that enable releasers
to provide a custom proxy host to connect to when performing the
promotion steps of a given release.

Signed-off-by: Ruy Adorno <ruyadorno@google.com>
ruyadorno force-pushed the add-host-env-var-to-release-script branch from 4fcdf24 to 9ce6067 Compare December 29, 2022 02:46

Copy link
Copy Markdown
Collaborator

aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 1, 2023
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 1, 2023
nodejs-github-bot merged commit 361d34c into nodejs:main Jan 1, 2023

Copy link
Copy Markdown
Collaborator

Landed in 361d34c

RafaelGSS pushed a commit that referenced this pull request Jan 2, 2023
Adds a `NODEJS_RELEASE_HOST` environment variable that enable releasers
to provide a custom proxy host to connect to when performing the
promotion steps of a given release.

Signed-off-by: Ruy Adorno <ruyadorno@google.com>
PR-URL: #45913
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
RafaelGSS mentioned this pull request Jan 2, 2023
ruyadorno deleted the add-host-env-var-to-release-script branch January 3, 2023 15:42
RafaelGSS pushed a commit that referenced this pull request Jan 4, 2023
Adds a `NODEJS_RELEASE_HOST` environment variable that enable releasers
to provide a custom proxy host to connect to when performing the
promotion steps of a given release.

Signed-off-by: Ruy Adorno <ruyadorno@google.com>
PR-URL: #45913
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Jan 5, 2023
Adds a `NODEJS_RELEASE_HOST` environment variable that enable releasers
to provide a custom proxy host to connect to when performing the
promotion steps of a given release.

Signed-off-by: Ruy Adorno <ruyadorno@google.com>
PR-URL: #45913
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
juanarbol pushed a commit that referenced this pull request Jan 26, 2023
Adds a `NODEJS_RELEASE_HOST` environment variable that enable releasers
to provide a custom proxy host to connect to when performing the
promotion steps of a given release.

Signed-off-by: Ruy Adorno <ruyadorno@google.com>
PR-URL: #45913
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
juanarbol mentioned this pull request Jan 28, 2023
juanarbol pushed a commit that referenced this pull request Jan 31, 2023
Adds a `NODEJS_RELEASE_HOST` environment variable that enable releasers
to provide a custom proxy host to connect to when performing the
promotion steps of a given release.

Signed-off-by: Ruy Adorno <ruyadorno@google.com>
PR-URL: #45913
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
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. tools Issues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants


Back | FazBrowse Home | New Git URL