Move fast and break things.
Merge vulnerabilities.
Double the work.
Merge code without tests.
Anything, but don’t let code become stale.

  • @[email protected]
    link
    fedilink
    English
    661 year ago

    Having a hard time determining whether this is sarcasm or not. Then I see the phrase “JavaScript Engineer” and become doubly confused.

      • @[email protected]
        link
        fedilink
        English
        28
        edit-2
        1 year ago

        I distinguish four types. There are clever, hardworking, stupid, and lazy officers. Usually two characteristics are combined. Some are clever and hardworking; their place is the General Staff. The next ones are stupid and lazy; they make up 90 percent of every army and are suited to routine duties. Anyone who is both clever and lazy is qualified for the highest leadership duties, because he possesses the mental clarity and strength of nerve necessary for difficult decisions. One must beware of anyone who is both stupid and hardworking; he must not be entrusted with any responsibility because he will always only cause damage.

        – Kurt von Hammerstein

        LinkedIn is Facebook for that last type.

      • Aviandelight
        link
        fedilink
        81 year ago

        That’s a relief because I thought I’d stumbled into LinkedIn Lunatics for a hot second.

        • terrrmus
          link
          fedilink
          English
          91 year ago

          Linkedin is for lunatics. Just a bunch of goobers giving digital handjobs to each other.

      • @[email protected]
        link
        fedilink
        41 year ago

        That could still be trolling. But LinkedIn is so full of utter garbage like this that it’s believable

        • @[email protected]
          link
          fedilink
          11 year ago

          I don’t think so. I just made a screenshot of one random convo he’s having about this, but there’s loads more in a similar fashion.

          And all of his other posts besides this one seem legit on the surface.

          So it would be pretty weird if he randomly has a very bad take, and then just claims “Lol this was a troll post, gotcha!”… That’s pretty much the 4chan defense when you get called out - “Haha guys, I’m actually not r-worded, I’m just trolling!”

      • @[email protected]
        link
        fedilink
        41 year ago

        Wow, of course he’s pretending the response is a misrepresentation of his opinion instead of defending it in good faith.

  • Danny M
    link
    fedilink
    291 year ago

    If somebody actually did that it would be grounds for removing their privileges to merge into master. THIS, THIS is why the JavaScript ecosystem has gotten so bad, people with mentalities similar to his.

  • @[email protected]
    link
    fedilink
    261 year ago

    Having to go through the process of merging hurts morale and slows performance. Give everyone on your team the right to force push to master.

  • @[email protected]
    link
    fedilink
    251 year ago

    I dunno but xtreme programming sounds like something straight outta Musk’s wettest teenage day dreams.

    • Bakkoda
      link
      fedilink
      2
      edit-2
      1 year ago

      Imagine if you will: You have a red button and a green button. You are allowed 10 seconds to review the code before rejecting or accepting & merging. Think fast.

  • @[email protected]
    link
    fedilink
    221 year ago

    This is satire, right? Surely no one would put their name on that publicly?

    Like someone working in a kitchen boasting about a life hack of not wasting time with hygiene.

  • @[email protected]
    link
    fedilink
    201 year ago

    Before everyone loses their minds, in Extreme Programming there are safeguards other than PR reviews. Before you submit a PR, you are supposed to have written the tests and to have written your code with pair programming, so your code already has some safety measures in place. On top of that, when you merge and deploy, more tests are run, and only if all of them are green do your changes go into production.

      • @[email protected]
        link
        fedilink
        English
        11 year ago

        Yes, that’s part of the point. Dumping all at once into a merge and asking people to comprehend it all isn’t particularly realistic.

    • agilobOP
      link
      fedilink
      English
      51 year ago

      you are supposed to have written the tests and to have written your code with pair programming,

      I commented out the tests because they were failing, pipelines were green so I merged. Now it’s running on prod. What do you do?

    • MeanEYE
      link
      fedilink
      31 year ago

      You lost me at “pair programming”. Having tests for what you can test is fine. But there’s code that simply can’t be tested, or at least not easily at which point you are just wasting time. Open source mantra is always great in my opinion… release early, release often. In addition to that have a test version of your software before you push it to production if there’s sensitive data. That’s usually good enough to catch issues.

      And he’s right, reviewing changes before merge just takes time and resources away from project while the master branch keeps moving. Merge, if there are issues, whoever submitted the change is obliged to fix it. You can always checkout earlier version.

      • @[email protected]
        link
        fedilink
        21 year ago

        I just made a github action that merges anything updated in master into feature branches automatically. you get pinged if there’s a conflict but the automerge keeps drift to a minimum so it’s less common and fixed sooner.

        better than merging poorly tested/reviewed code.

        and yeah, a small team of superstars doesn’t need reviews so much, but most teams have a range of devs with different levels of experience and time working with particular parts of a large codebase. Someone more senior or more expert derisks people picking up tickets and improves code quality.

        it also leads to plenty of good conversations about the best way to implement, so overall it’s a win.

        • MeanEYE
          link
          fedilink
          21 year ago

          Well, Git was designed to branch out, not be a single repo with bunch of users. So one team can have a local repo, that in turn gets merged into big one, etc. Structure matters as you say. Small experienced teams move fast. Big teams require a lot of management and supervision. I still think it’s better to split people up into small teams and give individual tasks, or let them pick tasks that need to be done.

  • @[email protected]
    link
    fedilink
    191 year ago

    I really wish LinkedIn would add an anonymous cringe emoji. I would use it on like 90% of the content on that site.

  • @[email protected]
    link
    fedilink
    English
    151 year ago

    Kinda acceptable if you have a slow release cadence. Everything needs to be reviewed and fixed/accepted (with defect/US raised) before production though.

    Needs to be in a smaller team with decent Devs too though!