add `From<GlobError> for io::Error` impl
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- rust
- Área
- backend-api-design
Línea de trabajo
Empieza por localizar GlobError y su implementación existente del formateo de errores, y compárala después con la conversión a io::Error solicitada en el issue. Verifica que la conversión preserve el mensaje contextual existente y que las funciones que devuelven io::Error puedan usar errores de glob con ?. Confirma que la suite de pruebas relevante pasa.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The conversion I would suggest is:
impl From<GlobError> for io::Error {
fn from(err: GobError) -> io::Error {
io::Error::new(err.error.kind(), err)
}
}
This will keep the "pretty context" so that glob errors keep printing as:
"attempting to read `{}` resulted in an error: {}",
self.path.display(),
self.error
Which is already very ergonomic.
I can do this if you OK it 😄
Benefits / Justification
The glob error is already essentially an io::Error with a bit of additional context. This will allow functions that return io::Error can use glob with ? directly which incurs (basically) no performance issues.
- Lenguaje dominante
- Rust
- Estrellas
- 597
- Forks
- 97
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 rust-lang/glob
-
[1.0] Change `MatchOptions::default` to match `MatchOptions::new`Posiblemente ocupada @Dym03 la tomó hace 380 días. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 64/100
-
`?` matches forward slashesAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 70/100
-
Make PatternToken publicAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
-
[1.0] Remove some trait implsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
Todos los issues de rust-lang/glob
Issues similares
-
good first issue help wanted program
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
hffgzktbcc-blip/ounce-core#297 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
nautechsystems/nautilus_trader#5631 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 69/100
bytecodealliance/rustix#1736 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
longbridge/gpui-kit#3871 ·
Los mantenedores suelen responder en 1 día
-
area:models enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día