this post was submitted on 24 May 2024
6 points (100.0% liked)

Programmer Humor

32558 readers
448 users here now

Post funny things about programming here! (Or just rant about your favourite programming language.)

Rules:

founded 5 years ago
MODERATORS
 
top 9 comments
sorted by: hot top controversial new old
[–] tills13@lemmy.world 2 points 6 months ago (1 children)

It's an internal inside of React.

[–] morrowind@lemmy.ml 0 points 6 months ago (1 children)

What am I looking at here?

[–] ImplyingImplications@lemmy.ca 0 points 6 months ago (1 children)

There's a string stating that the code should not be passed or the employee will be fired. I'd assume this was a test to see if an employee meant to be doing code review was actually doing them. Spoiler, they were not, as OP said they found this in production code.

[–] xmunk@sh.itjust.works 0 points 6 months ago (1 children)

I'd go a step further and I suspect it's a peppering string (i.e. fixed string you add to hashes to defeat rainbow tables). I'd really hope it isn't as you mentioned because gosh that sounds like a toxic workplace if someone is just leaving landmines around purely to get someone fired.

[–] skulblaka@startrek.website 1 points 6 months ago (1 children)

More like, you know damn well that Jim keeps passing code reviews without reading a line in them, he's been talked to, still does it, and you need something actionable to prove it so that you can get someone's ass in his chair who does their job.

[–] jcg@halubilo.social 1 points 6 months ago

From the stories I've heard from corporate software employees, this does sound like exactly the kind of thing you gotta do to show some manager the guy is buddy-buddy with that they're actually not doing their job. And even then they didn't listen.

[–] xmunk@sh.itjust.works 0 points 6 months ago (1 children)

What an absolutely shitty peppering string.

The code file has 32.171 lines, at least after beautifing it. 1MB fucking js code.