Groups should be NonEmptySeqs
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- fsharp
- Área
- tooling
Línea de trabajo
Comienza revisando las funciones de agrupación existentes, especialmente chunkBy, junto con el comportamiento de groupBy en F# core. Determina qué APIs de agrupación podrían devolver NonEmptySeq y evalúa las preocupaciones indicadas sobre la compatibilidad hacia atrás y la conversión ascendente de seq. Se considera terminado cuando el alcance de la API y el enfoque de compatibilidad estén acordados e implementados.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
When grouping sequences typically through a groupBy operation, all subsequences we get are never empty.
This means that using a NonEmptySeq should be more appropriate and it will provide additional compile time guarantees for further operations, typically aggregation operations.
The problem is that these grouping operations are defined in F# core and it's against this library design guidelines to redefine a function coming from F# core, for good reasons.
But we could redefine our grouping functions, for instance chunkBy will never yield an empty group. Here the problem is backwards compatibility and the additional burden incurred in having to eventually upcast to seq for certain operations. But for F# 6 it shouldn't be an issue since it does it automatically.
- Lenguaje dominante
- F#
- Estrellas
- 942
- Forks
- 106
- 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 fsprojects/FSharpPlus
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
fsprojects/FSharpPlus#683 · 1 comentario · 2 reacciones ·
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
fsprojects/FSharpPlus#678 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
fsprojects/FSharpPlus#644 ·
-
breaking change F#+ v2.0
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
fsprojects/FSharpPlus#643 ·
-
lens type signatureAbiertobreaking change F#+ v2.0
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
fsprojects/FSharpPlus#641 ·
Todos los issues de fsprojects/FSharpPlus
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
zerocracy/judges-action#2729 ·
Los mantenedores suelen responder en 8 días
-
agent/scanner bug hive/hosted-available-lke648397-260827-5n31
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Add `django-upgrade` to the CIAbiertodependencies feature github_actions good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
wemake-services/wemake-django-template#3149 ·
Los mantenedores suelen responder en 1 día
-
`Processing lsp` never exits and leaves orphaned processesPosiblemente ocupada @overcast302 la tomó hoy. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
processing/processing4#1578 · 1 comentario ·
-
autoInject recall silently drops memory injection on long / non-Latin prompts (HTTP 400 Query too long)Posiblemente ocupada @Epsilon006 la tomó hoy. Abiertointegration:coding-agents
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
vectorize-io/hindsight#5476 · 1 comentario ·
Los mantenedores suelen responder en 1 día