GitActions - testowanie wykonywalności plików

unkn0w / noobs

Kolekcja skryptów do szybkiego stawiania usług na serwerach Mikrusa

MIT License

251 stars 64 forks source link

GitActions - testowanie wykonywalności plików #135

Closed loper closed 1 year ago

loper commented 2 years ago

Czy w takim GitActions da się dodać krok sprawdzający, czy nowy skrypt ma mode 0755? To by pilnowało, żeby wszystko było wykonywalne. @mikasjp

mikasjp commented 2 years ago

Czy dobrze rozumiem, że chcesz, żeby Git pamiętał uprawienia plików ze skryptami? Poczytałem trochę na ten temat i wydaje mi się, że to nie jest dobry pomysł.

Git nie ma takiej możliwości "out of the box". Wymaga to dodatkowej konfiguracji.
Co jeśli ktoś pisze skrypt i używa Gita pod innym OS, który ma odmienny system uprawnień (np. Windows)?
Czy nie lepiej zmodyfikować skrypt https://noobs.mikr.us w taki sposób, żeby po sklonowaniu repo nadawał odpowiednie uprawnienia?

loper commented 2 years ago

Chciałbym, aby jako 1 z kroków testowania było wykonywane sprawdzanie, czy plik posiada tryb wykonywalny.

mikasjp commented 2 years ago

Chyba nie do końca rozumiem, co masz na myśli. Podczas testów są nadawane odpowiednie uprawnienia: https://github.com/unkn0w/noobs/blob/53f65269cf53f2d1cccc374be230106f6b10e69e/tests/execute_new_and_modified_scripts.sh#L16 Zaraz po sklonowaniu repozytorium skrypty nie będą wykonywalne.

loper commented 2 years ago

Scenariusz:

User dodaje nowy skrypt.
Zapomina jednak o tym, co większość - o nadaniu flagi wykonywalności (+x).
Test w Actions wyłapuje taki błąd i od razu sypie błędem.

mati97xdd commented 2 years ago

https://github.com/mati97xdd/noobs/commit/2f155df1298c7dc389fc921cb7e807cbd2d1a9f8 Tak będzie ok?

loper commented 2 years ago

Z samego czytania kodu - dokładnie o coś takiego mi chodziło. Da się to podpiąć do obecnych testów? Dzięki, L.