• TunaCowboy@lemmy.world
    link
    fedilink
    arrow-up
    8
    ·
    8 months ago

    You should not be pushing into your main/master/whatever branch.

    All the main/master replies completely miss the point, further emphasizing sirsirsalot’s statement.

    • Hawk@lemmynsfw.com
      link
      fedilink
      arrow-up
      2
      arrow-down
      1
      ·
      8 months ago

      I think I’ll keep doing it. It’s worked fine for the past decade 🤷

      • Solemn@lemmy.dbzer0.com
        link
        fedilink
        English
        arrow-up
        1
        ·
        8 months ago

        It works fine for small projects. I think that with more than 2-3 devs a PR based strategy works better for enforcing review and just makes life easier in general, since you end up with less stuff like force pushes to fix minor things like whitespace errors that break everyone’s local.