29
How large pull requests can exacerbate complexity and slow down development (graphite.dev)
submitted 2 years ago by to c/programming@programming.dev
10 comments fedilink hide all child comments

all 10 comments

sorted by: hot top controversial new old
[–] 27 points 2 years ago (2 children)

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

  • permalink
  • fedilink
  • source
  • hideshow 4 child comments
    •
  • [–] 1 point 2 years ago (1 child)

    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.

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 2 child comments
    •
  • [–] 1 point 2 years ago

    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.

  • permalink
  • fedilink
  • source
  • parent
    • [–] 10 points 2 years ago (1 child)

    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

  • permalink
  • fedilink
  • source
  • hideshow 2 child comments
    •
    • [–] 9 points 2 years ago

    Everyone remember to downvote ads

  • permalink
  • fedilink
  • source
    •
    [–] 2 points 2 years ago (1 child)

    The "review after merge" guy isn't looking quite so insane now. Stacking PRs? Really?

  • permalink
  • fedilink
  • source
  • hideshow 2 child comments
    •