Integrate updating/checking the config visualizations in the CI
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 25/100
Direzione di ricerca
Esamina gli script di visualizzazione e controllo di #514, il workflow CI e le aggiunte all'hook pre-commit a cui fanno riferimento i commit fd9a639 e 1e0aea5. Determina come rendere riproducibili i file DOT e PNG generati, quindi ripristina i controlli in CI e pre-commit in modo che le modifiche vengano rilevate senza falsi avvisi.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
In #514, I added config visualizations and a script that generates them. I had also added a script that checks if the DOT files were updated after running the visualization script again and which was triggered in the CI.
I had also added the update and check in the pre-commit hook, but since the precice-config-visualizer at that time required additional dependencies for the GUI, which was not possible to disable at that time, decided to not add this to the pre-commit in the end.
After getting an easy-to-install variant of the precice-config-visualizer, we can add this back into the pre-commit.
We I also disagreed with @fsimonis on how to implement the checks in the workflow. While I am also not happy having the automatically generated DOT files in the repository, and while we could generate them in the website CI, I would still like to keep the PNG files in the tutorials repository, because then the README files remain readable offline and on GitHub. The question would be if we can make them reproducible, so that we don't get false alerts to update them.
Commit that removed the checking and CI part: https://github.com/precice/tutorials/pull/514/commits/fd9a639849764ad76932613bf6ce5a88b11f3798
Commit that removed the pre-commit additions: https://github.com/precice/tutorials/pull/514/commits/1e0aea57438b208a13cbb5c94cc656017cfa3eef
- Lingua principale
- C
- Stelle
- 143
- Fork
- 169
- Merge medio
- 1g 11h
- PR unite (30g)
- 20
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 precice/tutorials
-
good first issue
Difficoltà 2/5 1-2 giorni Idoneità per principianti 62/100
precice/tutorials#617 · 2 commenti · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
system-tests
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
I maintainer di solito rispondono entro 1 giorno
-
system-tests
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
precice/tutorials#933 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 66/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 42/100
precice/tutorials#877 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di precice/tutorials
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 63/100
Xinyuan-LilyGO/T-Display-S3#352 · 3 commenti ·
-
documentation severity/minor
Difficoltà 1/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
docs: AGENTS.md has no rule against running deploy scripts into the owner's game installForse già presa @thesprockee l’ha presa oggi. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
EchoTools/nevr-runtime#524 ·
I maintainer di solito rispondono entro 1 giorno
-
python library: btc_sign() sends antiklepto host nonce for taproot miniscript (tr() policy) inputs because is_taproot() only recognizes simple P2TRForse già presa @benma-agent l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
BitBoxSwiss/bitbox02-firmware#2145 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 74/100
Genymobile/scrcpy#7097 ·
I maintainer di solito rispondono entro 1 giorno