Planned replacement of tslint with eslint

U012PCVT2DB

Hello spryker-team,

i have noticed, that spryker will replace tslint with eslint in the next version (CC-25963 Replaced tslint with eslint for YVES. ).
Upon reviewing it for our current development and future updates, i saw the changes in package.json.

I don't think this is an optimal solution, since this only binds the linting to the PYZ-namespace and no custom namespace.
Another suggestion would be to rename the yves:eslint function to just "yves:lint" so any future linter-updates won't break pipelines and functionalities.

Can you share some insight on why you chose to bind this only to PYZ or has this maybe slipped through your internal review?

Yuriy Gerton

Hello Sven.
Thank you for good point 🤝
It’s named according to the previous version.
We’ll rename it to the variant without the linter name.

Yuriy Gerton

https://github.com/spryker-shop/suite/pull/1236/files

U012PCVT2DB

My only concern (now) is that is only checking the pyz-namespace and no additional one 😞

before it checked every namespace.

Yuriy Gerton

We expect that the linter must only check project-level YVES components.
According to the folder structure.
Each project can customize the paths according to its requirements.