you are viewing a single comment's thread
view the rest of the comments
[–] 23 points 6 months ago (5 children)

The pr reviewers have nightmares about

  • source
  • hideshow 5 child comments
  • [–] [S] 18 points 6 months ago (3 children)

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

  • source
  • parent
  • hideshow 3 child comments
  • [–] 3 points 6 months ago (2 children)

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

  • source
  • parent
  • hideshow 2 child comments
  • [–] [S] 2 points 6 months ago (1 child)

    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

  • source
  • parent
  • hideshow 1 child comment
  • [–] 2 points 6 months ago

    😆 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. 👍

  • source
  • parent