alanshaw/david

Use npm-registry-client instead of npm

Open

#33 aperta il 20 nov 2013

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (81 fork)batch import
help wanted

Metriche repository

Star
 (962 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Since you're only using npm.commands.view it would be better to just use something like:

var RegClient = require('npm-registry-client')
var client = new RegClient(config)

client.get("npm", "latest", 1000, function (er, data, raw, res) {
  // error is an error if there was a problem.
  // data is the parsed data object
  // raw is the json string
  // res is the response from couch
})

npm-registry-client is the library depended on by npm to fetch things from the registry. Why is this better?

  1. Reduces dependencies: As a library about dependencies this should be self-explanatory :-D
  2. More configurable: If I wanted to use david to check against two different registries I am currently limited by what is on disk in .npmrc when you call npm.load

I would make a PR for this if you're interested.

Guida contributor

Direzione di ricerca
Innanzitutto, individua dove `npm` è attualmente utilizzato (probabilmente tramite `npm.commands.view`) nella codebase, ad esempio in un modulo che controlla le versioni dei pacchetti. Quindi, sostituisci tale utilizzo con `npm registry client` come mostrato nell'issue. Assicurati che il client sia istanziato con una configurazione appropriata (ad esempio da `.npmrc` o da opzioni passate). Aggiorna di conseguenza eventuali test. Il file principale del repository è `index.js` o `lib/david.js`.
Tech stack
javascriptnodejs
Dominio
backendtooling
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70