Trzasnąłem kolejnego refactora

General
,
1

Hej, uporządkowałem jeszcze bardziej repo sealiousowe (sealious-archive) w https://hub.sealcode.org/rSARbc5fee044d610bd93b4cc25c233996977363edae

Jest tam mnóstwo zmian, głównie związanych z przenoszeniem kodu, więc nie robiłem do tego review, tylko dałem commita na alphę.

Podsumowanie commita:

  • powrót do rozwiązania z jednym głównym katalogiem w repo (podział na podkatalogi bardzo przeszkadzał npm-owi, zob. https://github.com/npm/npm/issues/2974)
  • testy nadal wykonują się w dockerze
  • wywaliłem stare testy jednostkowe
  • przekopiowałem ConfigManagera 1-do-1 z Sealious Rewrite
  • nieudane połączenie z mongo już nie failuje po cichu
1 Like
2

Kolejny dobry krok :slight_smile: Jaka jest teraz strategia testowania? Bo chyba nie widzę w repo tej testowej appki.

Do tego teraz wiemy, jak Phabricator radzi sobie z wielkimi zmianami - “This commit is very large. Load each file individually.”

3

Testy do single_reference znajduja sie w https://hub.sealcode.org/diffusion/SAR/browse/alpha/lib/app/base-chips/field-types/single_reference.subtest.js - na potrzebe kazdego subtestu jest tworzona tymczasowa apka :wink:

4

Na początku jak zaciągnąłem zmiany to był trochę szok, ale ostatecznie łatwo się było w tym odnaleźć.

1 Like