Update mkdirp dev dep
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript, node.js
Línea de trabajo
Lee test/write.test.js y compara sus llamadas a mkdirp con el parche basado en promesas propuesto. Verifica que las pruebas de escritura pasen con la dependencia de desarrollo mkdirp actualizada y que los errores sigan manifestándose como se espera.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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;
});
});
});
- Lenguaje dominante
- C++
- Estrellas
- 101
- Forks
- 42
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de mapbox/node-zipfile
-
npm install构建失败 Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 20/100
mapbox/node-zipfile#87 ·
-
Passing password on file Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
mapbox/node-zipfile#86 ·
-
Fail to build with Node.js ≥ 12 Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
mapbox/node-zipfile#83 · 2 reacciones ·
Todos los issues de mapbox/node-zipfile
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
duckdb/duckdb-wasm#2258 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
objectionary/eo-graphs#75 ·
-
Coarray integration tests carry no LABELS, so run_tests.py silently skips them under every backend Abiertocoarray
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
FISCO-BCOS/FISCO-BCOS#5642 ·