search

kestra-io / kestra

Orchestration and automation platform to execute millions of scheduled and event-driven workflows declaratively in code and from the UI
https://kestra.io
Apache License 2.0
9.1k stars 589 forks source link

Add a UI linter for usage of `this` in vue template #5181

Open loicmathieu opened 2 days ago

loicmathieu commented 2 days ago

Issue description

Using this in vue template works when running the apps via npm run dev but not when build.

We have a lot of these issues, like https://github.com/kestra-io/kestra/pull/5180 so we need to add a linter for that so CI fail.

Byt the way, InteliJ warn about it!

yuri1969 commented 2 days ago

The ESLint Vue plugin apparently doesn't treat the this keyword in Vue templates as something leading to unintended behavior but as a mere stylistic problem. This is why it wasn't caught by the current linter setup.