• 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.