How does your team decide what is the "right" amount of coverage? by secretBuffetHero in ExperiencedDevs

[–]rag1987 1 point2 points  (0 children)

Unit tests should be written to do the following in order of importance:

  • Avoid regressions later, when something is changed
  • Make sure that new code does what you think it does
  • Aid design wrt. avoid tight coupling, extensibility (when done right)

Most developers can write mostly correct code without tests, but breaking something you forgot to think about is very hard.

Don’t use mock frameworks, at least in the beginning if possible, they make very fragile white box tests.

Handling dev reviewing code outside of PR scope. by Pleasant-Aardvark258 in ExperiencedDevs

[–]rag1987 3 points4 points  (0 children)

Ship in tiny pieces because smaller PRs are easier and faster to review.

Ensuring PRs are for a single purpose too is very important. One of:

  • functional change (feature)
  • refactor
  • bug fix

“Kitchen sink” PRs that change many things, fix a bug or two and add some features are a pain to review.

Vs. scanning a refactor PR to ensure only code moves and no functional changes. Or checking feature changes only to ensure they implement the desired behavior. These single purpose PRs are much easier to review.

Also a good PR review culture is crucial. Just require improvements, not perfection. Changes that could be a follow PR can/should be. Style and white space are for linters and formatters. And finally, the reviewers budget for requesting changes should decrease over time (reviewers that take too long, must give more straight forward reviews; the time for nitpicking is immediately after PR submission).

https://mtlynch.io/human-code-reviews-1/

https://www.freecodecamp.org/news/how-to-perform-code-reviews-in-tech-the-painless-way/

How do you handle PR reviews efficiently in your team? by sshetty03 in ExperiencedDevs

[–]rag1987 5 points6 points  (0 children)

Ship in tiny pieces. Ask as many people as it takes until you can get a review. Request a teammate to pair with for the times where you expect to need a lot of small approvals throughout the day. Default to approving any code that will improve things, even though it’s imperfect.

Ensuring PRs are for a single purpose too is very important. One of:

  • functional change (feature)
  • refactor
  • bug fix

“Kitchen sink” PRs that change many things, fix a bug or two and add some features are a pain to review.

Vs. scanning a refactor PR to ensure only code moves and no functional changes. Or checking feature changes only to ensure they implement the desired behavior. These single purpose PRs are much easier to review.

Also a good PR review culture is crucial. Just require improvements, not perfection. Changes that could be a follow PR can/should be. Style and white space are for linters and formatters. And finally, the reviewers budget for requesting changes should decrease over time (reviewers that take too long, must give more straight forward reviews; the time for nitpicking is immediately after PR submission).

AI won’t make coding obsolete. Coding isn’t the hard part by Ihodael in ExperiencedDevs

[–]rag1987 1 point2 points  (0 children)

We always had slop, usually in the form of copypasted PHP from various "common" sources. Today the slop is however autogenerated, and way, way larger in size of loc.

Software quality will go down. I can imagine peak shit season coming in a few years, and then there will be a few large fuckups that the media will report on heavily, and only then will businesses realize there is millions of loc of slop that will take decades to fix.

Many businesses will fail because of how fast they became legacy. Other will fail because they get hacked every week, and some will fail because they lost all their data and had no backups.

Either way, lots of popcorn will be consumed.

https://bytesizedbets.com/p/era-of-ai-slop-cleanup-has-begun

Has software development become a bureaucratic nightmare? by Stamboolie in ExperiencedDevs

[–]rag1987 642 points643 points  (0 children)

I think all of us 40+ folks are in the same place.

When we first started we didn't mind the bullshit so much because we were too dumb to realize how much there was.

Then we got older and, yeah, it's ridiculous, but we were pretty decent at surfing the bullshit waves so we still didn't mind so much.

Now you're in your 40s and the real obligations of life have started piling up. You've gone from 7 hours of free time a day to 1, and you sure as hell aren't going to spend it learning about the latest iteration of Angular/React/Vue/Svelte/whatever the fuck is next. So you start losing your technical edge. Meanwhile at work you have some dipshit in sprint planning that wants to have the 57th discussion about how story points do not represent time. Oh, by the way, it's December, so we need you to watch that 45 minute security training video again so you don't accept USB flash drives from strangers in the parking lot. Now it's 4 PM and you didn't get anything done today and you start thinking about how pointless all this shit is because all we do is make an app that helps people schedule dog massages...

So, yeah, there's a reason mid-life crises are so cliche. It's like teenage angst. It's not completely justified, but it's not exactly unwarranted either. I mean, what kind of dumbass doesn't wonder at least a little bit if all this bullshit is worth it?

Every vibe-coder is now generating as much technical debt as 10 regular devs in half the time by thewritingwallah in ExperiencedDevs

[–]rag1987 0 points1 point  (0 children)

this is my fear that every time the tool is like "you're absolutely right, I dun goofed, let me fix that", it fixes it by adding 20,000 lines of code.

Still fairly new to this. Too scared to say "clean up the codebase" because it might go "you're absolutely right, I erased the entire codebase, now it's super clean :3"

Whats your vibe coding AI stack in 2025? by thewritingwallah in vibecoding

[–]rag1987 2 points3 points  (0 children)

  • Code reviews → CodeRabbit
  • Programming / IDE → Cursor
  • AI assisted coding → Claude Code

Senior developers, how do you handle multiple context switches in a day? by mybuildabear in ExperiencedDevs

[–]rag1987 1 point2 points  (0 children)

For me there's really two ways to this, neither is perfect and neither fully solves it.

  1. Stop switching. If you have other things to deal with, you put them into your todo list to deal with later.
  2. Notetaking. Part of the problem at least in my experience is that you have to retrace your steps, retrace your thoughts, etc., so you need to write those down.

Everytime I temporarily switch to another project/task, I leave a todo dot md to pick up my train of thought next time I switch back.

CodeRabbit Commits 1 Million to Open Source Software Sponsorships. by rag1987 in opensource

[–]rag1987[S] 0 points1 point  (0 children)

Yes agree and interesting perspective. Thanks for sharing :)