How large pull requests slow down development
graphite.dev
external-link
One of the fundamental challenges in software engineering is managing and minimizing complexity. This challenge is not just a theoretical concern; it has real and tangible impacts on the pace of development.

Reads more like a page designed to funnel you into their product IMO

Imagine if a trumpet cleaning supply company wrote up an article about the importance of cleaning your trumpet regularly, and somewhere in the article they mentioned something like, ‘oh and you can use your trumpet cleaning spray to clean your trumpet’. Does this mean the information is incorrect? No? Any more or less valid than some dude named Craig on YouTube showing you how he cleans his trumpet? I don’t think so?

Yeah it’s an ad, but it’s an informative ad from an organization that obviously has some passion for the area of their product. I think it would actually be kind of weird if a company that does version control had absolutely no documentation on what version control best practices are.

Best practices for minimizing complexity:

  1. Try out “stacking”
  2. Simplify software design

I didn’t say there wasn’t information in there but the above paraphrased quote goes to the heart of what my comment was about.

Firstly, how is purchasing their product considered a “best practice”? It’s not generally accepted or the standard superior option by any stretch of the imagination.

Secondly, the option they give to minimizing complexity is to simplify your software design. Ignoring a couple problems with this statement, if they’re being honest this should be above the recommendation to “try out stacking”.

It doesn’t have to be that deep. You can give it a quick read and take from it what you will, but it is an ad for their product more so than it is an article that contains broadly useful information. They have every right to do so and maybe their product really is tremendously great but I’m just calling it how I see it.

Paradox
link
fedilink
English
1010M

Graphite is ok, but honestly it’s a solution in search of a problem

Maybe if you have a massive pr, splitting it up like this works, but that’s really a planning failure. Stories should be smaller, and if you need to keep them separate for a long time, use feature branches

Feature flags are another solution. Merge the code, but have it disabled on prod until you enable the feature flag (when the whole feature is complete and deployed)

Paradox
link
fedilink
English
210M

I’ve only ever worked in one codebase that didn’t need feature flags, and even then we could have used them.

DroneRights [it/its]
link
fedilink
English
910M

Everyone remember to downvote ads

The “review after merge” guy isn’t looking quite so insane now. Stacking PRs? Really?

What’s the story behind this? It sounds interesting.

Create a post

Welcome to the main community in programming.dev! Feel free to post anything relating to programming here!

Cross posting is strongly encouraged in the instance. If you feel your post or another person’s post makes sense in another community cross post into it.

Hope you enjoy the instance!

Rules

Rules

  • Follow the programming.dev instance rules
  • Keep content related to programming in some way
  • If you’re posting long videos try to add in some form of tldr for those who don’t want to watch videos

Wormhole

Follow the wormhole through a path of communities !webdev@programming.dev



  • 1 user online
  • 1 user / day
  • 1 user / week
  • 1 user / month
  • 1 user / 6 months
  • 1 subscriber
  • 1.21K Posts
  • 17.8K Comments
  • Modlog