ul-fmf / projekt-tomo

Spletna storitev za poučevanje programiranja
https://www.projekt-tomo.si
GNU Affero General Public License v3.0
14 stars 23 forks source link

Continuous integration #260

Closed matijapretnar closed 1 year ago

matijapretnar commented 1 year ago

Da se lahko osredotočimo na vsebino, sem naredil GitHub akcijo, ki požene black in isort, oba s privzetimi možnostmi. Tu je oblikovana koda, ki prestane vse teste. Kakšne pripombe?

nikerzetic commented 1 year ago

V lint.yaml je zahtevana najnovejša verzija black, v requirements pa fiksna verzija. Bo Github Actions samodejno posodablja requirements?

Sicer pa, glede na to, da je to samo linter, to ni tako pomembno.

matijapretnar commented 1 year ago

Pa res. V resnici nima smisla, da je black naštet v requirements.txt. Načeloma ga ima lahko uporabnik nameščen tudi kot ločen binary, tako da vključevanje lahko povzroči samo zmedo. Sem pobrisal.

matijapretnar commented 1 year ago

Uspelo mi je usposobiti še samodejno testiranje kode in posodobiti stare teste, da gredo vsi čez.

mucamaca commented 1 year ago

Malo pozno, ampak morda bi bilo bolje zaganjati CI s Postgresom namesto z Sqlite, ker se zaradi drobnih razlik med bazami med razvijanjem vcasih zgodijo kaksni problemi, ki jih CI potem lahko ujame.

matijapretnar commented 1 year ago

Se strinjam! Ko bo master nadgrajen na najnovejši Django, bi testiranje uredil prek Dockerja.