Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

feat(rome_cli): Add JS runtime and package manager information to rage #4021

Merged
merged 1 commit into from
Dec 9, 2022

Conversation

MichaReiser
Copy link
Contributor

@MichaReiser MichaReiser commented Dec 9, 2022

Summary

This PR extends the rome rage command to include the name and version of the JavaScript runtime that executed the rome wrapper script as well as the name and version of the package manager:

Test Plan

❯ npx rome rage
CLI:
  Version:              0.0.0
  Color support:        true

Platform:
  CPU Architecture:     x86_64
  OS:                   windows

Environment:
  ROME_LOG_DIR:         unset
  NO_COLOR:             unset
  TERM:                 unset
  JS_RUNTIME_VERSION:   "v16.13.2"
  JS_RUNTIME_NAME:      "node"
  NODE_PACKAGE_MANAGER: "npm/8.3.0"

 ❯ yarn rome rage
 yarn run v1.22.19
 $ C:\Users\Micha\git\rome-test\node_modules\.bin\rome rage
 CLI:
   Version:              0.0.0
   Color support:        true

 Platform:
   CPU Architecture:     x86_64
   OS:                   windows

 Environment:
   ROME_LOG_DIR:         unset
   NO_COLOR:             unset
   TERM:                 unset
   JS_RUNTIME_VERSION:   "v16.13.2"
   JS_RUNTIME_NAME:      "node"
   NODE_PACKAGE_MANAGER: "yarn/1.22.19"

❯ pnpm rome rage
CLI:
  Version:              0.0.0
  Color support:        true

Platform:
  CPU Architecture:     x86_64
  OS:                   windows

Environment:
  ROME_LOG_DIR:         unset
  NO_COLOR:             unset
  TERM:                 unset
  JS_RUNTIME_VERSION:   "v16.13.2"
  JS_RUNTIME_NAME:      "node"
  NODE_PACKAGE_MANAGER: "pnpm/7.13.6"

@MichaReiser MichaReiser added the A-CLI Area: CLI label Dec 9, 2022
@MichaReiser MichaReiser requested review from leops, ematipico and a team as code owners December 9, 2022 11:26
@netlify
Copy link

netlify bot commented Dec 9, 2022

Deploy Preview for docs-rometools ready!

Name Link
🔨 Latest commit 145756d
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/63931c310bd4b300080414de
😎 Deploy Preview https://deploy-preview-4021--docs-rometools.netlify.app/playground
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

## Summary

This PR extends the `rome rage` command to include the name and version of the JavaScript runtime that executed the `rome` wrapper script as well as the name and version of the package manager:

## Test Plan

```
❯ npx rome rage
CLI:
  Version:              0.0.0
  Color support:        true

Platform:
  CPU Architecture:     x86_64
  OS:                   windows

Environment:
  ROME_LOG_DIR:         unset
  NO_COLOR:             unset
  TERM:                 unset
  JS_RUNTIME_VERSION:   "v16.13.2"
  JS_RUNTIME_RELEASE:   "node"
  NODE_PACKAGE_MANAGER: "npm/8.3.0"

 ❯ yarn rome rage
 yarn run v1.22.19
 $ C:\Users\Micha\git\rome-test\node_modules\.bin\rome rage
 CLI:
   Version:              0.0.0
   Color support:        true

 Platform:
   CPU Architecture:     x86_64
   OS:                   windows

 Environment:
   ROME_LOG_DIR:         unset
   NO_COLOR:             unset
   TERM:                 unset
   JS_RUNTIME_VERSION:   "v16.13.2"
   JS_RUNTIME_RELEASE:   "node"
   NODE_PACKAGE_MANAGER: "yarn/1.22.19"

❯ pnpm rome rage
CLI:
  Version:              0.0.0
  Color support:        true

Platform:
  CPU Architecture:     x86_64
  OS:                   windows

Environment:
  ROME_LOG_DIR:         unset
  NO_COLOR:             unset
  TERM:                 unset
  JS_RUNTIME_VERSION:   "v16.13.2"
  JS_RUNTIME_RELEASE:   "node"
  NODE_PACKAGE_MANAGER: "pnpm/7.13.6"
```
@ematipico ematipico merged commit 2d1a0ef into main Dec 9, 2022
@ematipico ematipico deleted the feat/add-node-to-rage branch December 9, 2022 12:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-CLI Area: CLI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants