Shutting down removes comments from config.yml
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 55/100
Línea de trabajo
The issue identifies Warehouse.saveConfig and the shipped config.yml; start by locating that method and tracing what calls it during startup and shutdown. Check where default stores are set and saved, then verify that starting and stopping the server leaves config.yml unchanged.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
Every clean shutdown rewrites config.yml and removes its comments, including the shipped notes about default stores and custom messages. Warehouse.saveConfig saves the config with a YAML loader that doesn't keep comments.
Confirmed by loading and saving the shipped config.yml: the file had comments before and none after.
Enabling comments in the loader won't keep commented-out example lines, because those aren't attached to any setting.
Scope
- Only write
config.ymlwhen something changed it, such as setting a default store. Better still, move runtime values like default stores into store data, so the plugin never rewrites the admin's file.
Done when
- Starting and stopping the server leaves
config.ymlunchanged.
- Lenguaje dominante
- Java
- Estrellas
- 1
- Forks
- 0
- Merge medio
- 2 h 35 min
- PR fusionados (30 d)
- 14
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la 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 BrendonButler/Shops
-
dependencies enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
BrendonButler/Shops#79 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
BrendonButler/Shops#70 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
BrendonButler/Shops#69 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
BrendonButler/Shops#68 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
BrendonButler/Shops#67 ·
Los mantenedores suelen responder en 1 día
Todos los issues de BrendonButler/Shops
Issues similares
-
hermes-queue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 91/100
chrisalunlloyd2-sudo/mindpalace#373 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
bug priority: lowest
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
SkriptLang/Skript#8953 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
quality-assurance
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Paul-Austin-Oswego-CSC480-HCI521/gift-app#228 ·
Los mantenedores suelen responder en 1 día
-
[java] TestClassWithoutTestCases: document that abstract classes and interfaces are not checkedAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 91/100
Los mantenedores suelen responder en 2 días