dthree/vorpal

[for discussion] move default localStorage location under $HOME

Open

#122 ouverte le 1 mars 2016

Voir sur GitHub
 (11 commentaires) (3 réactions) (0 assignés)JavaScript (313 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (5 609 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

i ran into a problem when trying out your cash project. With my nodejs/npm setup, global installs require sudo: sudo npm i -g cash

however, because of that, cash doesn't have permissions to write to its own localstorage directory:

$ cash
fs.js:794
  return binding.mkdir(pathModule._makeLong(path),
                 ^

Error: EACCES: permission denied, mkdir '/Users/tenaciousmv/npm/lib/node_modules/cash/node_modules/vorpal/dist/.local_storage'

obviously i don't want to run cash with sudo :)

isn't it more typical to put this kind of storage in ~/.vorpal or ~/.cash or somewhere else under $HOME?

Guide contributeur

Direction de recherche
Étudiez comment modifier le chemin par défaut du localStorage en $HOME/.vorpal et mettre à jour le code en conséquence. Assurez vous de gérer correctement les autorisations.
Stack technique
javascriptnodejs
Domaine
cli
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Node.jsnpm
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70