1654 Konwencja w uporządkowaniu elementów plików .vue

[AThit7]

Zmiany w plikach .vue zmieniają kolejność bloków <script>, <template>, i <style> tak, żeby były one zgodne z przyjętą konwencją. Dodanie do projektu nowego lintera (ESLint) ma na celu zachowanie tej konwencji w przyszłości. Linter jest uruchamiany zaraz po tym, gdy swoje działanie zakończy prettier (wierz 8. w pliku package.json)

Jak można było się spodziewać, dodanie nowego lintera poskutkowało wykryciem przez niego kilkuset fragmentów kodu, które są niezgodne z pewnymi zasadami jakości. Wyłączyłem wszystkie takie zasady i pogrupowałem je według najmniejszego, w sensie zawierania, zbioru reguł, do którego należą (patrz plik .eslintrc.js).

Jeśli chodzi o zbiory reguł dla Vue.js, to można przeczytać o nich tutaj (ja wykorzystałem "recommended") https://eslint.vuejs.org/rules/ lub dla wykorzystanej przez mnie wersji tu https://github.com/vuejs/eslint-plugin-vue/blob/v7.20.0/docs/rules/README.md.

Dla TypeScripta wykorzystałem zbiory reguł "recommended" i "recommended-requiring-type-checking". Można o nich (i o innych zbiorach reguł) przeczytać tutaj https://github.com/typescript-eslint/typescript-eslint/blob/v4.33.0/packages/eslint-plugin/src/configs/README.md.

[lgpawel]

Czy cokolwiek odwołuje się do tej definicji, skoro z playbooka wypadła linia yarn lint? Tj. poza oczywistym faktem, że możemy mieć ochotę sami użyć tego polecenia... A jeśli ta definicja zostaje, to jaka jest zaleta tego, że playbook z niej nie korzysta?

[AThit7]

Instrukcja zostaje właśnie po to, żeby ktoś mógł z niej skorzystać. Playbook z niej nie korzysta, bo decyduje on o poprawności wykonania zadania na podstawie exit code'a (czyli w naszym przypadku tylko exit code'a eslint --ext .vue .). Można by to polecenie zmienić na prettier --check . & eslint --ext .vue . ale bashowy operator & jest leniwy, a chyba chcemy za każdym razem uruchamiać oba polecenia. Rozbicie tego na dwa kroki w playbooku zwiększa też czytelność wypisywanych przez github informacji, jak widać na załączonym obrazku.

[AThit7]

Zmiany: teraz wykonanie się kroków "Format Node assets", "Lint Node assets" i "Lint Python" zależy tylko od poprawnego wykonia się ostatniego z kroków przygotowujących czyli "Install Node Dependencies".

Warunki wyglądają tak if: ${{ always() && ... }} ponieważ bez always() lub podobnej funkcji bedą się zachowywały tak jakby tam było success(). (https://docs.github.com/en/actions/learn-github-actions/expressions#status-check-functions)