fix(vendor_excluded): Removed exclude-pattern for vendor/vendor_prefixed directory

W sytuacji, gdy testuję kod znajdujący się w katalogu /vendor to testy nie działają. Wg mnie wykluczanie nie jest potrzebne, co w pliku głównym ktoś ustawia, jakie ścieżki mają być testowane. Musiałaby taka osoba świadomie podać tam katalog /vendor.

assigned to @grola, @dyszczo, @sebastian.pisula, @potreb, @marcinkolanko, and @bartjaskulski

By Mateusz Gbiorczyk on 2021-08-30T12:13:48 (imported from GitLab)

@mateuszgbiorczyk commitowałeś tylko changelog :)

W gruncie rzeczy nie ma problemu, tylko - po co testować vendor? Nie zdarzyło mi się jeszcze, żebym potrzebował czegoś takiego

By Bartek Jaskulski on 2021-08-30T12:38:31 (imported from GitLab)

Hahah... poprawione ;)

By Mateusz Gbiorczyk on 2021-08-30T12:38:31 (imported from GitLab)

added 1 commit

• da362d14 - fix(vendor_excluded): Removed exclude-pattern for vendor/vendor_prefixed directory

Compare with previous version

By Mateusz Gbiorczyk on 2021-08-30T12:32:40 (imported from GitLab)

Powtórzę po @bartjaskulski : po co sprawdzać vendor i vendor_prefixed?

By Grzegorz Rola on 2021-08-31T12:20:47 (imported from GitLab)

@grola Nie sprawdzać rzecz jasna, ale ten wyjątek sprawia, że gdy pracowałem nad biblioteką i była ona w katalogu /vendor wtyczki to testy PHPCS nie działały.

Czyli byłem w tym katalogu (i w tym katalogu odpalam polecenie composera uruchamiające testy): /wp-content/plugins/flexible-checkout-fields/vendor/wpdesk/wp-wpdesk-deactivation-modal

I ta reguła nie pomija tylko katalogów /wp-wpdesk-deactivation-modal/vendor/ oraz /wp-wpdesk-deactivation-modal/vendor_prefixed/, ale pomija katalogi wyżej, tj.:

• /wp-content/plugins/flexible-checkout-fields/vendor/
• /wp-content/plugins/flexible-checkout-fields/vendor_prefixed/

Ten wyjątek wg mnie nie jest potrzebny, ponieważ PHPCS nie sprawdza katalogu /vendor, gdy nie doda się go świadomie w konfiguracji. Domyślnie sprawdzany jest katalog /src i /templates. A ten wyjątek powoduje inne problemy opisane powyżej.

By Mateusz Gbiorczyk on 2021-08-31T12:33:48 (imported from GitLab)

Nad bibliotekami proponuję pracować inaczej. Ja mam biblioteki w oddzielnym katalogu i używam w PHPStorm funkcjonalności deployment: Tools -> Deployment, która automatycznie wgrywa mi zmieniane pliki do katalogu vendor we wtyczce. Wiem, że @dyszczo robi to jeszcze inaczej, używając lokalnych źródeł w composer.json

A ostatni argument do mnie przemawia :)

By Grzegorz Rola on 2021-08-31T12:33:48 (imported from GitLab)

Też używałem lokalnych źródeł, ale jako lokalne źródło wskazałem katalog w /vendor. Z pewnością dałoby się tam wskazać dowolny na dysku, ale tak zwyczajnie było mi wygodniej.

By Mateusz Gbiorczyk on 2021-08-31T12:42:10 (imported from GitLab)

approved this merge request

By Grzegorz Rola on 2021-08-31T12:33:52 (imported from GitLab)

unassigned @grola

By Grzegorz Rola on 2021-08-31T12:34:05 (imported from GitLab)

approved this merge request

By Dyszczo on 2021-08-31T13:16:37 (imported from GitLab)

approved this merge request

By Bartek Jaskulski on 2021-08-31T13:16:46 (imported from GitLab)

unassigned @bartjaskulski

By Bartek Jaskulski on 2021-08-31T13:16:54 (imported from GitLab)

unassigned @dyszczo

By Dyszczo on 2021-08-31T13:34:59 (imported from GitLab)

unassigned @potreb

By Piotr Po on 2021-09-03T07:06:45 (imported from GitLab)

unassigned @sebastian.pisula

By Sebastian Pisula on 2021-09-16T08:28:59 (imported from GitLab)

closed