Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Detect pending changes of front-end #913

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Detect pending changes of front-end #913

wants to merge 5 commits into from

Conversation

ad-m
Copy link
Member

@ad-m ad-m commented Sep 27, 2020

No description provided.

@ad-m ad-m marked this pull request as draft October 25, 2020 23:54
@ad-m
Copy link
Member Author

ad-m commented Oct 25, 2020

@rwakulszowa , może krokiem w kierunku testów front-endu jest doprowadzenie tego PR-a do końca? Wymaga upewnienia się (ręcznego), że wszystko po zbudowaniu jest takie jak powinno być (znasz incydent z fedrowania). Przy okazji można wypisać co mogłoby się zepsuć i co trzeba testować. Wówczas będziemy mogli – podobnie jak w Fedrowaniu – zaktualizować dużą część bibliotek front-end dość bezpiecznie. Nie ma po co testować front-endu, skoro front-end nie jest zmieniany, bo strach.

@rwakulszowa
Copy link
Member

Jasne, jak najbardziej. Rozumiem, że (w mocnym skrócie), po dodaniu tego wymagania, żeby przepchnąć jakąkolwiek zmianę trzeba najpierw zaktualizować frontend, tak?

Generalnie jestem za. Wczoraj budowałem lokalnie poradnię i faktycznie pojawiły się problemy, które rozwiązała aktualizacja zależności.

Czy publikacja na produkcję dzieje się automatycznie (np. cyklicznie), czy jest to ręczny proces?

@ad-m
Copy link
Member Author

ad-m commented Oct 26, 2020

Ręczny proces, ale zautomatyzowany do jednego polecenia.

@ad-m
Copy link
Member Author

ad-m commented Oct 26, 2020

Zmiana ma na celu zagwarantowanie, że stan źródeł dla front-endu odpowiada zbudowanemu front-endowi, skoro już go trzymamy w repozytorium. Zagwarantowanie tego nie powinno być trudne, bo build powinien być deterministyczny, zwłaszcza jak będziemy mieli dokładnie opisane zależności (istotny jest package-lock.json dla zależności zależności). W naszym bundlu front-endu mamy trochę rzeczy z paczek Python wyciągany, więc to nas zabezpieczy przed niepostrzeżonym zdesynchronizowaniem scommitowanego bundla przy okazji aktualiczki paczki Python / paczki npm / innej zależności front-end.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants