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

Do not depend on nodeJS version in example output #7695

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

MarkEWaite
Copy link
Contributor

Do not depend on nodeJS version in example output

A recent documentation feedback noted that the incorrect version number string is included in the sample output. Rather than adjust the example output, let's use example output that is unlikely to change between NodeJS releases. This example uses process.platform and process.env.CI. Those are not likely to change from one version of nodeJS to the next.

The feedback sheet includes the note that the NodeJS version number is incorrect in the example output.

A recent documentation feedback noted that the incorrect version number
string is included in the sample output.  Rather than adjust the example
output, let's use example output that is unlikely to change between NodeJS
releases.  This example uses `process.platform` and `process.env.CI`.
Those are not likely to change from one version of nodeJS to the next.

https://docs.google.com/spreadsheets/d/1nA8xVOkyKmZ8oTYSLdwjborT0w-BpBNNZT0nxR9deZ8/edit?usp=sharing
is the feedback sheet that included the note that the NodeJS version
number is incorrect in the example output.
@MarkEWaite MarkEWaite requested a review from a team as a code owner November 22, 2024 15:14
Copy link
Member

@krisstern krisstern left a comment

Choose a reason for hiding this comment

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

LGTM

@MarkEWaite MarkEWaite merged commit 17d8964 into jenkins-infra:master Nov 22, 2024
5 checks passed
@MarkEWaite MarkEWaite deleted the change-example-command branch November 22, 2024 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants