Tomasz-Zdeb / Embedded-Systems-Class-Project

Activity Project for Embedded Systems class in 2022/2023 winter semester at Cracow Uniwersity Of Technology part-time Computer Science studies
MIT License
0 stars 0 forks source link

Ograniczenie zasad akceptacji pull requestów #3

Closed Tomasz-Zdeb closed 2 years ago

Tomasz-Zdeb commented 2 years ago

Chciałbym abyśmy znaleźli sposób na ograniczenie możliwości akceptacji pull requestów, aby administrator mógł pilnować, czy ktoś nie próbuje przemycić na maina fuszerki 😀

jackpie2 commented 2 years ago

Jeśli poprzednie rozwiązania się nie sprawdziły, to jest to pewnie kwestia tego, że jesteśmy dodani jako kolaboranci do repo. Żeby już z tym nie mieszać, to chyba drugi najłatwiejszy sposób to utworzenie pliku CODEOWNERS w roocie (albo .gtihub/ lub docs/). Z tego co rozumiem, to wystarczy, że zawartością tego pliku będzie:

* @Tomasz-Zdeb

I wtedy w branch protection rules: Require a pull request before merging-> Require review from Code Owners

Można spróbować to przetestować na innym branchu najpierw, bo ten plik działa jedynie na swój branch.

Tomasz-Zdeb commented 2 years ago

Zrealizowałem powyższe instrukcje na masterze. Poproszę @BlessedSamhain o sprawdzenie, ponieważ stanowi osobę trzecią z punktu widzenia tego PR.


Poniższy komunikat pozwala mieć nadzieję na poprawne funkcjonowanie tego rozwiązania 😊 image

Tomasz-Zdeb commented 2 years ago

Przy pomocy @BlessedSamhain sprawdziłem obecne zachowanie mechanizmu. Wszystko działa zgodnie z planem. Dobra robota! @jackpie2 😀