Marko Kosir

Results 14 comments of Marko Kosir

Hey, was anybody able to come to the bottom of this 🤔 Since we are passing `handler` "directly" it doesn't matter which endpoint url gets passed `const response = await...

Great stuff @ronparkdev, would love to have this rule 🙇‍♂️ 🚀

Hey @ljharb, with `create-next-app` I initialized an [empty project](https://github.com/mkosir/example-lint) and added [`jsx-handler-names` lint rule](https://github.com/mkosir/example-lint/blob/main/.eslintrc#L15) to it. If you run `npm run lint` linting will succeed without errors, with [example component](https://github.com/mkosir/example-lint/blob/main/components/TestComponent.tsx)...

Thx @ljharb wouldn't figure it out. Maybe only one caveat worth mentioning in the docs would be that props should not be destructed as it needs `onClick={prop.onPropCallback}` to work, since...

> Props should always be destructured Not following completely 🤔 I updated the [test repo](https://github.com/mkosir/example-lint) mentioned above, if you try to run `npm run lint` linting fails on [this line](https://github.com/mkosir/example-lint/blob/main/components/TestComponent.tsx#L20)...

Yep it make sense to change rule defaults 👍 Thx for clarification on this one, unfortunately I wont be enabling this (otherwise awesome) rule, since props destructing is must have.

> you can enable it with the checkLocalNames option You probably ment `checkLocalVariables` 🤔 ? If thats the case I have already [enabled it](https://github.com/mkosir/example-lint/blob/main/.eslintrc#L15) and as mentioned [lint fails](https://github.com/mkosir/example-lint/blob/main/components/TestComponent.tsx#L20) (but...

Hm but [onShouldLint](https://github.com/mkosir/example-lint/blob/main/components/TestComponent.tsx#L20) is a prop name just destructed. Anyhow I just feel I own you a 🍺 IRL for the discussion I'm putting you through 😄

Hey, went through the answers and tried different approaches but nothing worked. My question is how it would be possible to run my shell/bash script in separate file (my-script.sh), which...

Any news or feedback on this? It's the only thing thats keeps me switching monorepo from pnpm to bun.