refactor rule

86 points · 6 comments · view on lemmy.world

6 Comments

WILSOOON@programming.dev · 23 pts · 103d (5 replies)

The pr reviewers have nightmares about

python@lemmy.world · 18 pts · 103d (3 replies)

no worries, I''m force pushing to main so there isn't any PR ☺️

victorz@lemmy.world · 2 pts · 103d (2 replies)

I hope you fix that exhaustive-deps violation as well! Mustn't leave any deps unexhausted!!

python@lemmy.world · 1 pts · 102d (1 reply)

Oh I know where the violation is, but fixing it in the way my linter wants me too actually breaks a lot of things. I played fast and loose with my useEffect hooks, and now I'm paying the price 🐎
Having completely fucked handmade code is still more fun than using AI tho, so I'm actually looking forward to untangling my mess

victorz@lemmy.world · 1 pts · 102d

πŸ˜† I hear that, I also enjoy fixing code.

Btw, if you have a useEffect where you don't want to use all its dependent variables, you might be interested in using other built-in reactive hooks. I could hook you up if you're interested. πŸ‘

four@lemmy.zip · 8 pts · 103d

LGTM πŸ‘