For a long time Firefox Desktop development has supported both Mercurial and Git users. This dual SCM requirement places a significant burden on teams which are already stretched thin in parts. We have made the decision to move Firefox development to Git.

  • We will continue to use Bugzilla, moz-phab, Phabricator, and Lando
  • Although we’ll be hosting the repository on GitHub, our contribution workflow will remain unchanged and we will not be accepting Pull Requests at this time
  • We’re still working through the planning stages, but we’re expecting at least six months before the migration begins

APPROACH

In order to deliver gains into the hands of our engineers as early as possible, the work will be split into two components: developer-facing first, followed by piecemeal migration of backend infrastructure.

Phase One - Developer Facing

We’ll switch the primary repository from Mercurial to Git, at the same time removing support for Mercurial on developers’ workstations. At this point you’ll need to use Git locally, and will continue to use moz-phab to submit patches for review.

All changes will land on the Git repository, which will be unidirectionally synchronised into our existing Mercurial infrastructure.

Phase Two - Infrastructure

Respective teams will work on migrating infrastructure that sits atop Mercurial to Git. This will happen in an incremental manner rather than all at once.

By the end of this phase we will have completely removed support of Mercurial from our infrastructure.

  • Lord_ToRA@lemmy.world
    link
    fedilink
    English
    arrow-up
    28
    arrow-down
    9
    ·
    1 year ago

    I agree that PRs are problematic, but that doesn’t make GitHub “trash”.

    Also, that dude is so obnoxious and really seems to like to make broad generalizations of his opinion like it’s fact.

    • Scorpion3869@lemmy.ml
      link
      fedilink
      arrow-up
      8
      arrow-down
      1
      ·
      edit-2
      1 year ago

      and really seems to like to make broad generalizations of his opinion like it’s fact.

      None of what Brodie said is baseless, even if some are more opinion than fact. He sight’s sources for a reason.

      • Lord_ToRA@lemmy.world
        link
        fedilink
        English
        arrow-up
        6
        arrow-down
        2
        ·
        1 year ago

        He sight’s sources for a reason.

        I’d hope so. It wouldn’t be good if they were blind.

        Anyway, I didn’t question the accuracy of what he said, just that his manner of delivery is obnoxious and portrays an attitude of self-importance. It feels like he’s yelling at the viewer with hostility.

        • Spectacle8011@lemmy.comfysnug.space
          link
          fedilink
          arrow-up
          5
          ·
          1 year ago

          Aussies tend to be quite direct. It’s basically our natural state. I get how it can be perceived as hostile, but I don’t actually think Brodie is very abrasive. He seems like a pretty relaxed guy.

        • Scorpion3869@lemmy.ml
          link
          fedilink
          arrow-up
          5
          arrow-down
          1
          ·
          1 year ago

          He’s literally talking to the viewer the same way the vast majority of YouTubers speak, he’s just Australian.

          • Lord_ToRA@lemmy.world
            link
            fedilink
            English
            arrow-up
            2
            arrow-down
            2
            ·
            edit-2
            1 year ago

            the same way the vast majority of YouTubers speak

            “Everyone else is doing it” is not a free pass. Absolutely none of the creators I watch speak like that, and I can list quite a lot. It’s obnoxious.

              • Lord_ToRA@lemmy.world
                link
                fedilink
                English
                arrow-up
                1
                arrow-down
                3
                ·
                1 year ago

                Why do either of those matter towards being obnoxious?

                Are all Australians obligated to be like that? Two years ago people were supposed to make videos acting obnoxious?

      • Rustmilian@lemmy.world
        link
        fedilink
        arrow-up
        10
        ·
        edit-2
        1 year ago

        Instead of using valid commit messages they just slap on “#42” which isn’t valid anywhere other than the GitHub web UI.

      • IAm_A_Complete_Idiot@sh.itjust.works
        link
        fedilink
        arrow-up
        3
        ·
        1 year ago

        Also, GitHub PRs atleast to me feel like they encourage reviewing changes by the total diff of the entire PR, and not each commit. I don’t want a slog of commits that don’t add any value - it just makes doing things like reverts more annoying. Stuff like Gerrit and phabricator enforce reviews by making you review individual commits / changes / whatever you want to call them and not branch diffs.

        • bamboo@lemm.ee
          link
          fedilink
          arrow-up
          1
          arrow-down
          1
          ·
          1 year ago

          GitHub has an option when merging a PR to “squash and merge”. This option squashes all of the commits on the PR branch into a single commit and cherry-picks it on top of the base branch. We use this by default in our open source projects at work. Most people are not gonna go through the effort of making a well defined patch series the way it would be required for a Linux kernel contribution. Most changes aren’t that big though and so it doesn’t really matter. Send as many commits as you want in the PR, I’ll just review the diff as a whole and squash it when I’m done. Workflows should adapt to user preference, not the other way, and this is a good example of that.

          • IAm_A_Complete_Idiot@sh.itjust.works
            link
            fedilink
            arrow-up
            3
            ·
            1 year ago

            How much of that is what GitHub encourages and how much of that is what Users prefer? Plenty of users seem to enjoy phabricator / Gerrit for code review in practice precisely because of their workflows.

            • bamboo@lemm.ee
              link
              fedilink
              arrow-up
              1
              ·
              1 year ago

              Well squash and merge isn’t default or pushed in any way. It’s an option, and we chose to enable it ourselves because that’s what works best for us. It’s what works well for many other projects too, which is why many choose to enable it instead of the default merge commit.

              • IAm_A_Complete_Idiot@sh.itjust.works
                link
                fedilink
                arrow-up
                1
                ·
                edit-2
                1 year ago

                Yeah, but phabricator and Gerrit are entirely separate workflows from GitHub, and a lot of people prefer that workflow because it leads to encouraging better histories and reviews. It helps you in getting rid of the “fixed typos” type of commits, while still letting you make larger PRs.

                GitHub obviously does let you keep a clean git history, but the code review workflow in GH just doesn’t encourage reviewing commits.

                • bamboo@lemm.ee
                  link
                  fedilink
                  arrow-up
                  1
                  ·
                  1 year ago

                  I think the idea here is that reviewing individual commits is irrelevant if the plan is just to squash it all down. Each PR corresponds to a single change on the main branch in the end, the fact there was a main commit followed by a half size “fixed typos” and “fixed bug” commits doesn’t actually matter since it will be blown away in the end. The process results in the same clean history with good individual commits on the main branch, just as if the user squashes those commits locally before pushing it up to the code review platform.

                  • IAm_A_Complete_Idiot@sh.itjust.works
                    link
                    fedilink
                    arrow-up
                    1
                    ·
                    edit-2
                    1 year ago

                    Right, but squashed commits don’t scale for large PRs. You could argue that large PRs should be avoided, but sometimes they make sense. And in the case where you do have a large PR, a commit by commit review makes a lot of sense to keep your history clean.

                    Large features that are relatively isolated from the rest of the codebase make perfect sense to do in a different branch before merging it in - you don’t merge in half broken code. Squashing a large feature into one commit gets rid of any useful history that branch may have had.