feature-sliced / eslint-config

🍰 Lint feature-sliced concepts by existing eslint plugins
https://npmjs.com/@feature-sliced/eslint-config
MIT License
120 stars 5 forks source link

LINT(INTEGRATION): Move rule-configs close to tests #35

Closed azinit closed 2 years ago

azinit commented 2 years ago

Description

   // As variant
   // @see https://github.com/effector/eslint-plugin/tree/master/rules/no-getState
   📁 rules/
        📁 layers-slices/
              index.js          // config of boundaries
              fixtures.js       // test fixtures (if need)
              index.test.js     // unit test of boundaries
              index.md          // docs of boundaries (in future)
        📁 public-api/

Reference

https://github.com/effector/eslint-plugin/tree/master/rules/no-getState

Krakazybik commented 2 years ago

@azinit будем разделять config / rules? или будем всё в одно складывать?

📁 src/
        📁 configs/
        📁 rules/
        📁 utils/
azinit commented 2 years ago

@Krakazybik

1) Ну у них как раз все связанное с конкретным правилом - в rules/ и лежит) 2) А если речь про то, что у них:


Т.е. при возможности - было бы классно эти папочки дополнительные выделить. При этом тут главное по задаче - это хранить ресурсы конкретного правила вместе (тесты, фикстуры, конфиг, дока)

Вдохновился примером с FECrew так сказать))