Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Update mkdirp dev dep

Aperta
#85 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
1/5
Tempo stimato
1-3 ore
Idoneità per principianti
35/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Ferma
Stack tecnologico
javascript, node.js
Ambito
testing, tooling

Direzione di ricerca

Leggi test/write.test.js e confronta le sue chiamate a mkdirp con la patch proposta basata sulle promise. Verifica che i test di scrittura passino con la dipendenza di sviluppo mkdirp aggiornata e che gli errori continuino a manifestarsi come previsto.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

This simple patch seems enough:

--- a/test/write.test.js
+++ b/test/write.test.js
@@ -25,8 +25,7 @@
             var dest2 = path.join(__dirname,'tmp', 'aync-copy',name);
             mkdirp.sync(path.dirname(dest));
             mkdirp.sync(path.dirname(dest2));
-            mkdirp(path.dirname(dest), function(err) {
-                if (err) throw err;
+            mkdirp(path.dirname(dest)).then(() => {
                 if (path.extname(name)) {
                     zf.readFile(name, function(err, buffer) {
                         if (err) throw err;
@@ -48,6 +47,8 @@
                         });
                     });
                 }
+            }).catch( err => {
+                throw err;
             });
         });
     });
Lingua principale
C++
Stelle
101
Fork
42
Nessuna PR unita negli ultimi 30g

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di mapbox/node-zipfile

Tutte le issue di mapbox/node-zipfile

Issue simili

Altre issue su C++

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.