Add new builtin for merging maps
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 35/100
Línea de trabajo
No se nombran archivos ni pruebas. Empieza revisando cómo los builtins existentes manejan los mapas y cómo se especifica concat; después determina si esto debería ser un nuevo builtin de merge/copy o una extensión de concat. Se considera terminado cuando el ejemplo del issue con campos anidados funciona con una semántica de merge documentada y las pruebas cubren el comportamiento.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Expr currently does not have a builtin for merging maps. Adding a new builtin would be useful to merge maps but also to modify nested fields.
For instance:
/*
Env:
{
"data":{
"status": 0,
"rows": 32,
"signing_keys":[
"key1",
"key2"
]
}
}
*/
merge($env["data"], {signing_keys: concat($env["data"]["signing_keys"], ["another"])})
/*
Output:
{
"data":{
"status": 0,
"rows": 32,
"signing_keys":[
"key1",
"key2",
"another"
]
}
}
*/
As for the name, "merge" seems natural but I would also consider using "copy" (as used in Go's maps package). Other possibility is to modify "concat" to support both arrays and maps.
- Lenguaje dominante
- Go
- Estrellas
- 8k
- Forks
- 535
- 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 expr-lang/expr
-
docs needed
Dificultad 1/5 1-3 horas Aptitud para principiantes 62/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
-
`find`, `findIndex`, `first` and `get` are type-checked as the element type where they return nilAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
-
Release latest master branchAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
Todos los issues de expr-lang/expr
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
kind/engineering pulumi/pulumi-terraform Task Workflow Failure
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
pulumi/pulumi-terraform#1215 ·
Los mantenedores suelen responder en 1 día
-
bug needs-acceptance wg/developer-experience-ecosystem
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
vllm-project/semantic-router#4480 ·
Los mantenedores suelen responder en 1 día
-
area/docs kind/documentation priority/backlog triage/accepted
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
lexfrei/cloudflare-tunnel-gateway-controller#943 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
keyxmakerx/Chronicle#967 ·
Los mantenedores suelen responder en 1 día