Merytorycznie baaardzo słabo.
Czemu? Przeklejam moje komentarze do jednego z artykułów:
Github to system kontroli wersji
Serio? Nie rozróżniasz Gita od Githuba?
Jeden komponent w jednym pliku, najlepiej żeby plik nie przekraczał 400 linii kodu.
Skąd ta liczba? Komponent powinien być zgodny z SRP, a nie jakąś liczbą linii z kosmosu.
Dlaczego w innych tutorialach wszyscy przypisują moduł do zmiennej? Jest to niepoprawne, a w przypadku gdzie mamy jeden moduł na plik wręcz zbędne.
O ile angularowym modułom wystarczy IFFE, to jak chcesz przetestować kod, którego moduły angularowe nie obejmują, a jest zamknięty w domknięciu? Module pattern to bardzo przydatny i ważny wzorzec, to co tutaj piszesz to jakieś herezje.
Brak przypisanej zmiennej poprawia czytelność kodu, oraz zapobiega przypadkowemu użyciu tej zmiennej ponownie.
A jak chcesz użyć jakiś fragmentu kodu ponownie?
Jeszcze jedna ogólna uwaga - używanie w nowych projektach standardu ES5 w 2017 roku to trochę taka archeologia. Już nie mówiąc o uczeniu tego innych.