idyll-lang/idyll

Issue with global version reporting

Open

#617 geöffnet am 26. Feb. 2020

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (1.977 Stars) (97 Forks)batch import
CLIRough Edgesbughelp wanted

Beschreibung

Describe the bug When running idyll --version in a directory outside of an idyll project, I see this:

$ idyll --version

  Using version undefined (installed globally).

  To install local dependencies in an Idyll project that
  you've cloned from GitHub, run `npm install`.

To Reproduce Steps to reproduce the behavior:

  1. Install the latest idyll globally (npm i -g idyll@latest)
  2. Navigate to a folder that does not contain an idyll project
  3. Run idyll --version
  4. See error

Expected behavior The correct version should be printed.

Contributor Guide

Tech Stack
javascriptnodejs
Domain
cli
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Keine
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80
Research-Richtung
The issue is that `idyll version` prints 'undefined' when run outside an Idyll project. The version is likely read from a local package.json, but falls back to global. Look at the CLI entry point file, e.g., bin/cli.js or similar, and see how it resolves the version. Possibly the global install's package.json path is not correctly resolved. A fix could use require('../package.json') or process.env.npm package version.