LINT-45: Split layers-slices-boundaries #46

Krakazybik · 2021-12-19T09:26:39Z

Description

Разбиты на отдельные конфиги slice и layers boundaries.
Пока сохранен slices-and-layers-boundaries (потом нужно будет написать мерж конфигов)
Добавлены тесты для slices-and-layers-boundaries.

Reference

#45

Checks

Description added
Self-reviewed
CI pass

azinit · 2021-12-19T16:48:38Z

Спс за крутецкое оформление PR! Описание прям в темплейт вынести можно ✊

azinit

Неприятно конечно с сплиттингом & мерджингом конфигов вышло... Но лан, главное что кастомайзинг худо бедно работает)

В ридмиху добавить бы инфу как минимум, а остальное - вопросы и некрит моменты

.prettierrc.js

index.js

rules/slices-and-layers-boundaries/index.js

rules/slices-and-layers-boundaries/index.test.js

index.js

rules/layers-boundaries/index.md

azinit · 2021-12-19T17:10:17Z

@Krakazybik Я кста не просто так в отдельную таску вынес сплиттинг layers и boundaries))

Т.к. на данном этапе это скорее больше проблем создает, и PR тоже разбухает

+ Я вспомнил, что мы рили вродь как договаривались, что забиваем на данный момент на сплиттинг

А делаем это отдельно по #45 задаче
(ее щас и не закроешь еще как минимум из-за того что не все требования выполнены) 🤷‍♂️

Krakazybik · 2021-12-19T17:21:48Z

@Krakazybik Я кста не просто так в отдельную таску вынес сплиттинг layers и boundaries))

Т.к. на данном этапе это скорее больше проблем создает, и PR тоже разбухает

Я вспомнил, что мы рили вродь как договаривались, что забиваем на данный момент на сплиттинг

А делаем это отдельно по #45 задаче (ее щас и не закроешь еще как минимум из-за того что не все требования выполнены) man_shrugging

окай =) оно просто уже поспличено было, значит будет как #45ый, 22ой то я проверил, там вроде бы всё?

azinit · 2021-12-19T17:47:21Z

окай =) оно просто уже поспличено было, значит будет как #45ый, 22ой то я проверил, там вроде бы всё?

Да, по 22-му тогда только в ридмиху инфу добавить бы как частично подключать
(даже с layers-slices-boundaries единым)

А потом отдельный PR на преттир+форматирование
И этот PR уже на LINT(CUSTOM): Add advanced customization of config #45

index.js

azinit · 2021-12-20T16:48:15Z

Кста если PR этот драфтовым хотел сделать - лучше рили добавить WIP: префикс в тайтле

А то уже ревьювить думал))

azinit · 2021-12-22T02:58:23Z

package.json

@@ -24,7 +24,6 @@
    "publish:patch": "npm version patch && npm publish",
    "publish:minor": "npm version minor && npm publish",
    "publish:major": "npm version major && npm publish",
-    "prettier:fix": "prettier --write **/*.js",


praise: Йеее 🔥🔥🔥🔥

azinit

Пингани пож, когда конфликты порезолвишь)

Krakazybik

Вроде как допилил и актуализировал.

Krakazybik · 2021-12-23T04:37:24Z

rules/integration.custom.js

+const assert = require("assert");
+const { configLib } = require("../utils");
+
+describe("Integration Custom configs tests:", () => {


Тестируем варианты кастмоного подключения правил.

Krakazybik · 2021-12-23T04:47:39Z

rules/integration.custom.js

+    })
+
+    describe("Without publicAPI: ", () => {
+


Без PublicAPI

Krakazybik · 2021-12-23T04:47:51Z

rules/integration.custom.js

+    })
+
+    describe("Without import-order: ", () => {
+


Без Import-order

Krakazybik · 2021-12-23T04:49:27Z

rules/integration.custom.js

+    })
+
+    describe("Without layers-slice, but with layers: ", () => {
+


С отдельно подключенным layers, на этом я думаю достаточно и отдельно проверять slices не имеет смысла, т.к. layers и slices имеют одинаковое правило под собой и влияют только друг на друга.

Krakazybik · 2021-12-23T04:50:21Z

rules/layers-slices/index.test.js

+});
+
+describe("Slices and Layers config:", () => {
+


Проверяем layers-slices вместе.

Krakazybik · 2021-12-23T04:56:01Z

rules/slices/slices.test.js

+        configLib.mockImports(cfg),
+    ),
+});
+


Тестируем slices отдельно

Также тестируем отдельно и layers, но он выше помечен как moved

Krakazybik · 2021-12-23T10:28:59Z

Вливать сначало МЕНЯ!

azinit · 2021-12-23T19:09:53Z

Вливать сначало МЕНЯ!

Так это же вообще про 0.2.0 история вродь как, не?)

Krakazybik · 2021-12-23T19:19:46Z

Вливать сначало МЕНЯ!

Так это же вообще про 0.2.0 история вродь как, не?)

Окай =( снова конфликты разруливать :D

azinit · 2021-12-24T20:30:50Z

Да у меня честно говоря пока в целом сомнения, насколько стоит сплитить щас layers и slices, учитывая единственно возможную реализацию

Я мб даже бы этот PR на паузу поставил 🤔

azinit · 2021-12-24T20:31:06Z

(+ учитывая что с public-api щас те же самые проблемы появились)

Krakazybik added 5 commits December 18, 2021 17:47

feat: Prettier added.

0b3e381

refactor: split slices-layers.

11d8f39

feta: slice-and-layers config.

920660d

revert: merge config.

667f6b9

fix: slices rules. remove redundant helpers.

0d3967d

Krakazybik self-assigned this Dec 19, 2021

Krakazybik added the refactor label Dec 19, 2021

Krakazybik added 2 commits December 19, 2021 12:34

fix(doc): wrong examples in readme.

acce6cc

refactor(test): simplify some test cases.

156ea0a

Krakazybik requested a review from azinit December 19, 2021 09:39

Krakazybik linked an issue Dec 19, 2021 that may be closed by this pull request

LINT(GET-STARTED): Check customization of config #22

Closed

6 tasks

azinit requested changes Dec 19, 2021

View reviewed changes

Krakazybik removed a link to an issue Dec 19, 2021

LINT(GET-STARTED): Check customization of config #22

Closed

6 tasks

Krakazybik changed the title ~~LINT-22(REFACTOR): Split layers-slices-boundaries~~ LINT-45(REFACTOR): Split layers-slices-boundaries Dec 19, 2021

Krakazybik linked an issue Dec 19, 2021 that may be closed by this pull request

LINT(CUSTOM): Add advanced customization of config #45

Open

6 tasks

Krakazybik added 3 commits December 20, 2021 11:52

revert: prettier

36ba453

revert: use old rule naming

68756d9

fix: rule name in main config

5449338

azinit reviewed Dec 20, 2021

View reviewed changes

index.js Outdated Show resolved Hide resolved

Krakazybik changed the title ~~LINT-45(REFACTOR): Split layers-slices-boundaries~~ LINT-45(REFACTOR): WIP: Split layers-slices-boundaries Dec 20, 2021

Krakazybik changed the title ~~LINT-45(REFACTOR): WIP: Split layers-slices-boundaries~~ WIP: LINT-45(REFACTOR): Split layers-slices-boundaries Dec 20, 2021

Krakazybik requested a review from azinit December 20, 2021 20:43

Krakazybik closed this Dec 20, 2021

Krakazybik deleted the refactor/LINT-22-split-layers-and-slices-boundaries branch December 20, 2021 21:03

Krakazybik restored the refactor/LINT-22-split-layers-and-slices-boundaries branch December 20, 2021 21:04

Krakazybik reopened this Dec 20, 2021

azinit changed the title ~~WIP: LINT-45(REFACTOR): Split layers-slices-boundaries~~ LINT-45(REFACTOR): Split layers-slices-boundaries Dec 22, 2021

azinit changed the title ~~LINT-45(REFACTOR): Split layers-slices-boundaries~~ LINT-45: Split layers-slices-boundaries Dec 22, 2021

azinit reviewed Dec 22, 2021

View reviewed changes

azinit requested changes Dec 22, 2021

View reviewed changes

Krakazybik changed the title ~~LINT-45: Split layers-slices-boundaries~~ WIP: LINT-45: Split layers-slices-boundaries Dec 22, 2021

Krakazybik added 4 commits December 23, 2021 07:04

chore: actualize

582124d

test: integration tests for custom config variants.

3028d6b

revert: publicAPI and doc changes.

3c0a8f3

fix(doc): usage layers and slices custom rules.

a6a2df3

Krakazybik commented Dec 23, 2021

View reviewed changes

Krakazybik changed the title ~~WIP: LINT-45: Split layers-slices-boundaries~~ LINT-45: Split layers-slices-boundaries Dec 23, 2021

Krakazybik requested a review from azinit December 23, 2021 04:58

Krakazybik added 2 commits December 23, 2021 08:01

chore: rename to index.test.js

e69a7af

doc(readme): actualize for new rules.

8ed3029

Krakazybik mentioned this pull request Dec 23, 2021

GITHUB-46(template): PR creation template #56

Merged

3 tasks

azinit added the on-hold label Dec 24, 2021

Krakazybik marked this pull request as draft December 24, 2021 20:32

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

LINT-45: Split layers-slices-boundaries #46

LINT-45: Split layers-slices-boundaries #46

Krakazybik commented Dec 19, 2021 •

edited

Loading

azinit commented Dec 19, 2021

azinit left a comment

azinit commented Dec 19, 2021

Krakazybik commented Dec 19, 2021

azinit commented Dec 19, 2021

azinit commented Dec 20, 2021

azinit Dec 22, 2021

azinit left a comment

Krakazybik left a comment

Krakazybik Dec 23, 2021

Krakazybik Dec 23, 2021

Krakazybik Dec 23, 2021

Krakazybik Dec 23, 2021

Krakazybik Dec 23, 2021

Krakazybik Dec 23, 2021

Krakazybik Dec 23, 2021

Krakazybik commented Dec 23, 2021

azinit commented Dec 23, 2021

Krakazybik commented Dec 23, 2021 •

edited

Loading

azinit commented Dec 24, 2021

azinit commented Dec 24, 2021

		})

		describe("Without layers-slice, but with layers: ", () => {

LINT-45: Split layers-slices-boundaries #46

Are you sure you want to change the base?

LINT-45: Split layers-slices-boundaries #46

Conversation

Krakazybik commented Dec 19, 2021 • edited Loading

Description

Reference

Checks

azinit commented Dec 19, 2021

azinit left a comment

Choose a reason for hiding this comment

azinit commented Dec 19, 2021

Krakazybik commented Dec 19, 2021

azinit commented Dec 19, 2021

azinit commented Dec 20, 2021

Choose a reason for hiding this comment

azinit left a comment

Choose a reason for hiding this comment

Krakazybik left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Krakazybik commented Dec 23, 2021

azinit commented Dec 23, 2021

Krakazybik commented Dec 23, 2021 • edited Loading

azinit commented Dec 24, 2021

azinit commented Dec 24, 2021

Krakazybik commented Dec 19, 2021 •

edited

Loading

Krakazybik commented Dec 23, 2021 •

edited

Loading