4 ms·
I have seen the same mistake made in code created by humans. Many times, especially in react / typescript/ JavaScript, someone will forget to use a lambda. I f
by dalemhurley 2y ago
I have seen the same mistake made in code created by humans. Many times, especially in react / typescript/ JavaScript, someone will forget to use a lambda.
I felt the blog post failed to articulate the root cause of the issue and went straight to blaming ChatGPT.
When you rush and make large or non peer code reviewed commits to main it is going to happen.
The real issue was when you rush, take shortcuts and don’t adequately test and peer code review then errors will occur.
I would have imagined that a test that tried a few different signup options would have found the issue immediately.
- kfarr 2y agoYeah ChatGPT is a red herring -- it doesn't matter what generates the code, it's what you do with it.
- whoknowsidont 2y agoSurely current events explains why ChatGPT is topical?
- Dylan16807 2y agoYou can have topical red herrings.
- deleted 2y ago[deleted]
- dalemhurley 2y agoTopical or not, blaming ChatGPT is only scratching the surface. To be truly reflective, OP needs to dive into the real reason their code had this issue. It wasn’t using GPT, it was not having the controls in place.
- hedora 2y agoMy mental model for ChatGPT is that it’s an entry-level engineer that will never be promoted to a terminal level and will eventually be let go. However, this engineer can type infinitely fast, which means it might be useful if used very carefully. Anyway, letting such a person near financially important code would lead to similar issues, and in both cases, I’d question the judgment of the person that decided to deploy the code at all, let alone without much testing.
- vsuperpower2020 2y agoI want to reclaim the word engineer from Americans. Turns out everyone is an engineer, even the chatbots.
- TeMPOraL 2y agoThis is kind of how it works with Real Engineering™ and other licensed professions - it's the non-licensed people doing most of the grunt work, the PE/architect/licensed professional reviews and signs off on it. But then, by virtue of their signature, they're still on the hook for any problems.
- OptionOfT 2y agoThe code is written by ChatGPT, pushed, and subsequently reviewed by ChatGPT. /s (I hope)
- conductr 2y agoJust another day livin’ in the matrix
- crabmusket 2y agoThis sort of issue seems common in a few places. E.g. Vue's component props can have defaults, and woe betide you if you use a literal object or array as a default, instead of a function that returns an object or array. I'm surprised there was no lint rule for this case.