Changeset View
Standalone View
.eslintrc
- This file was added.
{ | |||||
"extends": "sealcode" | |||||
} | |||||
kuba-orlik: Pieczołowicie przerobiłem cały kod tak, aby nie robił awaitów w pętlach. `time make test`… | |||||
Done Inline ActionsJestem za tym, aby to włączyć na co najmniej 1. Pomoże to zapobiegać na poleganiu na tym, że funkcja zwraca undefined itp. Albo coś zwracamy w każdym scenariuszu, albo nie zwracamy nic kuba-orlik: Jestem za tym, aby to włączyć na co najmniej 1. Pomoże to zapobiegać na poleganiu na tym, że… | |||||
Done Inline ActionsKiedyś miałem GENIALNY pomysł, aby realizować zgłaszanie błędów strategii dostępu i field-type'ów za pomocą rzucania exceptions. Dlatego mamy dużo return Promise.reject("message"). Ustawmy to na 1, aby nie zapomnieć tego naprawić. kuba-orlik: Kiedyś miałem GENIALNY pomysł, aby realizować zgłaszanie błędów strategii dostępu i field… | |||||
Done Inline ActionsJa jestem na tak (żeby to ustawić na 2). Skąd wątpliwości? kuba-orlik: Ja jestem na tak (żeby to ustawić na 2). Skąd wątpliwości? | |||||
Done Inline ActionsTa zasada generuje ponad 60 errorów. Bardziej niż o jej słuszność, chodzi mi o to, czy nie wyrzucić tego do osobnego taska. Bo w wielu miejsach polegamy na undefined i wiąże się to z dużym refaktorem logiki. Można by ten task połączyć z wyeliminowaniem błędów z consistent-return :) bartosz-gordon: Ta zasada generuje ponad 60 errorów. Bardziej niż o jej słuszność, chodzi mi o to, czy nie… | |||||
Done Inline Actionsustawmy na 0 kuba-orlik: ustawmy na 0 | |||||
Done Inline Actionsjeżeli dodamy "functions": false to imho będzie OK kuba-orlik: jeżeli dodamy `"functions": false` to imho będzie OK | |||||
Done Inline ActionsNigdy nie powinniśmy blokować wątku. Moim zdaniem to koniecznie trzeba ustawić na 2. kuba-orlik: Nigdy nie powinniśmy blokować wątku. Moim zdaniem to koniecznie trzeba ustawić na `2`. | |||||
Done Inline Actionsimho może tak zostać (tylko trzeba zmienić "warning" na "warn" kuba-orlik: imho może tak zostać (tylko trzeba zmienić "warning" na "warn" | |||||
Done Inline ActionsJest ok, możemy ew. zwiększyć limit do 75 linijek. I wyłączyć tę zasadę dla plików *.subtest kuba-orlik: Jest ok, możemy ew. zwiększyć limit do 75 linijek. I wyłączyć tę zasadę dla plików *.subtest | |||||
Done Inline ActionsZmiana tego wymaga fundamentalnego przebudowania API Sealiousa. Zob. {T1147} kuba-orlik: Zmiana tego wymaga fundamentalnego przebudowania API Sealiousa. Zob. {T1147} |
Pieczołowicie przerobiłem cały kod tak, aby nie robił awaitów w pętlach. time make test zwraca bardzo przybliżone wyniki. Ale trzeba pamiętać o tym, że w testach nie robimy zapytań na wiele zasobów (co najwyżej na ~6), więc to może nie być miarodajne.
Myślę, że warto pamiętać o tym, że można robić pętle wydajniej. Dajmy na warning.