add config validation by default
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 45/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- cli
Direzione di ricerca
Inizia da src/BaseCommand.js, in particolare da getFullConfig e dalla gestione esistente dei flag, quindi esamina src/commands/app/pack.js e src/commands/app/deploy.js insieme ai risultati della ricerca nel repository per getFullConfig. Assicurati che i comandi ereditati propaghino tramite spread i flag di BaseCommand e che la convalida della configurazione sia attiva per impostazione predefinita, ma possa essere disabilitata; verifica il comportamento dei comandi interessati e i test esistenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Right now, config validation is not turned on by default and only certain commands turn it on, like the app pack command:
https://github.com/adobe/aio-cli-plugin-app/blob/4fb2a25f56050e31d7f444b80c6261ca609d688e/src/commands/app/pack.js#L43
Modify BaseCommand.getFullConfig to validate the app config by default - this means that all commands that call getFullConfig will validate the config by default, even if they don't use the whole config:
https://github.com/search?q=repo%3Aadobe%2Faio-cli-plugin-app%20getFullConfig&type=code
BaseCommand.getFullConfig:
https://github.com/adobe/aio-cli-plugin-app/blob/4fb2a25f56050e31d7f444b80c6261ca609d688e/src/BaseCommand.js#L147
Provide a shared flag in the BaseCommand to turn off config validation, just in case there are issues:
- The flag should be called
config-validationandallowsNo : true,default: true. - The
getFullConfigmethod should read the flag directly - every command that inherits from BaseCommand must spread the
BaseCommandflags
- Lingua principale
- JavaScript
- Stelle
- 28
- Fork
- 41
- Merge medio
- 5h 54m
- PR unite (30g)
- 4
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di adobe/aio-cli-plugin-app
-
app:pack aborts when Node writes process warnings to api-mesh stderrForse già presa @raj-prince587 l’ha presa 74 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
adobe/aio-cli-plugin-app#930 ·
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
adobe/aio-cli-plugin-app#924 · 1 commento ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
adobe/aio-cli-plugin-app#919 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
adobe/aio-cli-plugin-app#890 ·
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 28/100
adobe/aio-cli-plugin-app#881 · 1 commento ·
Tutte le issue di adobe/aio-cli-plugin-app
Issue simili
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
Riloox/hostkind-open#45 ·
-
Request: semgrepApertanew icon permissions in review
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
simple-icons/simple-icons#15093 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 83/100
module-federation/core#5205 ·
I maintainer di solito rispondono entro 1 giorno
-
遥测历史查询始终返回 HTTP 400，页面静默无结果Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
aikido security:critical
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 76/100
dequelabs/monitor-utility#63 ·