Details

Reviewers

kuba-orlik

Group Reviewers

Unknown Object (Project)

Maniphest Tasks

Unknown Object (Maniphest Task)

Summary

Add base eslint/prettier config

Do not use public/private as properties

Fix eqeqeq

Fix dot notation

Fix no-else-return

Fix strict

Fix prefer-template

Fix prefer-arrow-callback

Fix object-shorthand

Fix no-var

Fix no-lonely-if

Make eslint up to date

Diff Detail

Repository

rS Sealious

Branch

T1118 (branched from alpha)

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 956
Build 956: arc lint + arc unit

Event Timeline

bartosz-gordon created this revision.Sep 30 2018, 11:29

Checklista diffów do sealiousa:

EJKO działa poprawnie (make test + testy manualne) - jeżeli nie, zrobić odpowiedniego diffa do EJKO
sealious-template działa poprawnie (make test)- jeżeli nie, zrobić odpowiedniego diffa do sealious-template
aktualizacja dokumentacji
czy wszystko nadal się zgadza w getting_started

bartosz-gordon requested review of this revision.Sep 30 2018, 11:29

Harbormaster completed remote builds in B956: Diff 1291.Sep 30 2018, 11:29

bartosz-gordon planned changes to this revision.Sep 30 2018, 11:29

Fix no-negated-condition
Remove unused variables
No redundant await
Radix
no-undef
Consistent callbacks

Harbormaster completed remote builds in B957: Diff 1292.Sep 30 2018, 13:26

bartosz-gordon planned changes to this revision.Sep 30 2018, 13:26

no-prototype-builtin
line between class members
return in callback
class-methods-use-this
no-throw-literal
no-useless-escape
no-shadow
prefer-spread

Harbormaster completed remote builds in B966: Diff 1302.Oct 3 2018, 16:50

bartosz-gordon planned changes to this revision.Oct 3 2018, 16:50

no-params-reassign

Harbormaster completed remote builds in B968: Diff 1304.Oct 3 2018, 18:59

bartosz-gordon retitled this revision from Run fixable rules to ESLint config and error fixes.Oct 3 2018, 19:00

bartosz-gordon edited the test plan for this revision. (Show Details)

@kuba-orlik Chciałbym, żebyś zerknął na zakomentowane linie oznaczone jako TO DISCUSS i każdą z nich odkomentował i puścił npm run lint. Wtedy możemy ocenić, czy warto zmieniać te rzeczy w Sealiousie :)

bartosz-gordon added a reviewer: Unknown Object (Project).Oct 4 2018, 09:19

O ile pamiętam, to planowaliśmy wrzucić ten konfig jako reużywalny sealcode'owy - tak, aby w .exlintrc wystarczyło podać "sealcode" - czy planujesz to zrobić w ramach tego diffa?

.eslintrc
22	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`.
61	Jestem 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
121	Kiedyś 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ć.
137	Ja jestem na tak (żeby to ustawić na 2). Skąd wątpliwości?
139	jeżeli dodamy `"functions": false` to imho będzie OK
150	Nigdy nie powinniśmy blokować wątku. Moim zdaniem to koniecznie trzeba ustawić na `2`.
183	imho może tak zostać (tylko trzeba zmienić "warning" na "warn"
184	Jest ok, możemy ew. zwiększyć limit do 75 linijek. I wyłączyć tę zasadę dla plików *.subtest
186	Zmiana tego wymaga fundamentalnego przebudowania API Sealiousa. Zob. {T1147}
191	ustawmy na 0

kuba-orlik requested changes to this revision.Oct 4 2018, 18:46

This revision now requires changes to proceed.Oct 4 2018, 18:46

Zapomniałem o tym, zrobię to ;)

.eslintrc
137	Ta 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` :)

Refactor file manager
Refactor file manager
Assign tasks to eslint rules

Harbormaster completed remote builds in B1010: Diff 1356.Oct 10 2018, 18:19

bartosz-gordon planned changes to this revision.Oct 10 2018, 18:19

Use shareable eslint-config-sealcode

Harbormaster completed remote builds in B1011: Diff 1357.Oct 10 2018, 19:31

bartosz-gordon edited the test plan for this revision. (Show Details)Oct 10 2018, 19:33

Przejrałem pliki wyrywkowo i nie znalazałem nieprawidłowości. Testy przechodzą, program do zarządzania magazynami działa (zarówno TEST jak i POPULATE)

This revision is now accepted and ready to land.Oct 11 2018, 12:10

kuba-orlik added a project: Sealious.Oct 11 2018, 12:11

kuba-orlik changed the visibility from "All Users" to "Public (No Login Required)".

ESLint config and error fixesAbandonedPublicActions

Details

Diff Detail

Event Timeline

Large Diff

Revision ContentsChangeset List

Diff 1291

.eslintrc

.prettierrc

lib/action.js

lib/app/app.js

lib/app/base-chips/access-strategy-types/and.js

lib/app/base-chips/access-strategy-types/and.subtest.js

lib/app/base-chips/access-strategy-types/logged_in.js

lib/app/base-chips/access-strategy-types/noone.js

lib/app/base-chips/access-strategy-types/not.js

lib/app/base-chips/access-strategy-types/not.subtest.js

lib/app/base-chips/access-strategy-types/or.js

lib/app/base-chips/access-strategy-types/or.subtest.js

lib/app/base-chips/access-strategy-types/owner.js

lib/app/base-chips/access-strategy-types/public.js

lib/app/base-chips/access-strategy-types/same-anon.js

lib/app/base-chips/access-strategy-types/same-as-for-resource-in-field.js

lib/app/base-chips/access-strategy-types/same-as-for-resource-in-field.subtest.js

lib/app/base-chips/access-strategy-types/super.js

lib/app/base-chips/access-strategy-types/themselves.js

lib/app/base-chips/access-strategy-types/user-referenced-in-field.js

lib/app/base-chips/access-strategy-types/user-referenced-in-field.subtest.js

lib/app/base-chips/access-strategy-types/users-who-can.js

lib/app/base-chips/access-strategy-types/when.js

lib/app/base-chips/calculated-field-types/aggregate.js

lib/app/base-chips/calculated-field-types/custom.js

lib/app/base-chips/calculated-field-types/map-reduce.js

lib/app/base-chips/collections/formatted-images.js

lib/app/base-chips/collections/password-reset-intents.subtest.js

lib/app/base-chips/collections/users.js

lib/app/base-chips/field-types/boolean.js

lib/app/base-chips/field-types/cached-value.js

lib/app/base-chips/field-types/color.js

lib/app/base-chips/field-types/context.js

lib/app/base-chips/field-types/control-access.js

lib/app/base-chips/field-types/control-access.subtest.js

lib/app/base-chips/field-types/date.js

lib/app/base-chips/field-types/datetime.js

lib/app/base-chips/field-types/derived-value.js

lib/app/base-chips/field-types/email.js

lib/app/base-chips/field-types/enum.js

lib/app/base-chips/field-types/file-reference.js

lib/app/base-chips/field-types/file.js

lib/app/base-chips/field-types/float.js

lib/app/base-chips/field-types/html.js

lib/app/base-chips/field-types/image.js

lib/app/base-chips/field-types/int.js

lib/app/base-chips/field-types/json-object.js

lib/app/base-chips/field-types/password.js

lib/app/base-chips/field-types/reverse-single-reference.js

lib/app/base-chips/field-types/secret-token.js

lib/app/base-chips/field-types/settable-by.js

lib/app/base-chips/field-types/single_reference.js

lib/app/base-chips/field-types/text.js

lib/app/base-chips/field-types/username.js

lib/app/base-chips/field-types/value-existing-in-collection.js

lib/app/base-chips/field-types/value-exists-in-collection.js

lib/app/chip-manager.js

lib/app/file-manager.js

lib/app/load-base-chips.js

lib/app/logger.js

lib/app/metadata.js

lib/app/run-action-curry.js

lib/chip-types/access-strategy-type.js

lib/chip-types/access-strategy.js

lib/chip-types/aggregated-field.js

lib/chip-types/calculated-field-type.js

lib/chip-types/channel.js

lib/chip-types/chip.js

lib/chip-types/collection.js

lib/chip-types/datastore.js

lib/chip-types/field-structures.js

lib/chip-types/field-type-default-methods.js

lib/chip-types/field-type.js

ESLint config and error fixes
AbandonedPublic
Actions

Revision Contents
Changeset List