install.sh: instance size menu does not pre-select the default size
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 84/100
Línea de trabajo
Lee la lógica de selección del tamaño en install.sh, líneas 2556–2583, especialmente el cálculo de default_type y la llamada gum choose --selected. Asocia cada tipo de instancia compatible con su etiqueta de opción completa y comprueba que la selección predeterminada coincida con el tamaño mínimo que necesita el pack. El trabajo estará listo cuando el tamaño requerido esté preseleccionado y pulsar Enter no despliegue una instancia demasiado pequeña.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
_gum_or_die size_choice $GUM choose --header "Instance size" \
--selected "${default_type} — recommended" \
"t4g.medium — 2 vCPU, 4GB ~\$25/mo light use" \
"t4g.large — 2 vCPU, 8GB ~\$50/mo regular use" \
"t4g.xlarge — 4 vCPU, 16GB ~\$100/mo recommended" || size_choice=""
Problem: gum choose --selected must get the full option text. The text "t4g.xlarge — recommended" is not equal to any option, so no option is pre-selected and the cursor stays on t4g.medium. If the user pushes Enter, the installer deploys t4g.medium, also when the pack needs a larger minimum size (L2556-2572).
Fix: Map default_type to its full option label and pass that label to --selected. For example:
local sel
case "$default_type" in
t4g.medium) sel="t4g.medium — 2 vCPU, 4GB ~\$25/mo light use" ;;
t4g.large) sel="t4g.large — 2 vCPU, 8GB ~\$50/mo regular use" ;;
*) sel="t4g.xlarge — 4 vCPU, 16GB ~\$100/mo recommended" ;;
esac
Found by a code read at commit 1e50295. The code was read, but not run on a real instance.
- Lenguaje dominante
- Shell
- Estrellas
- 73
- Forks
- 11
- Merge medio
- 6 h 46 min
- PR fusionados (30 d)
- 5
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 inceptionstack/lowkey
-
bug good first issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
inceptionstack/lowkey#135 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 94/100
inceptionstack/lowkey#133 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
inceptionstack/lowkey#131 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
inceptionstack/lowkey#130 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
inceptionstack/lowkey#121 ·
Los mantenedores suelen responder en 1 día
Todos los issues de inceptionstack/lowkey
Issues similares
-
[cifuzz] "Minimizing crashes reduces fuzzing time" is logged for every batch run, even with minimize-crashes offPosiblemente ocupada @martinfrancois la tomó hoy. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
alunduil/alunduil-chezmoi#914 ·
Los mantenedores suelen responder en 1 día
-
cao-evolution cao-evolution:compiler-security
Dificultad 1/5 Menos de una hora Aptitud para principiantes 86/100
githubnext/gh-aw-cao#17586 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día