Hello,

I have only ever used git in very very basic way: commit, branch, merge request, that’s pretty much it.

I have a use case where I pull the repository locally, branch (let’s call it branch1), write some code, test it, commit, then create a merge request to master branch.

The merge request takes some time to be approved. During that time I would like to add more edits on top of those I submitted in the merge request. What would be the correct steps here?

  • mryessir@lemmy.sdf.org
    link
    fedilink
    arrow-up
    2
    ·
    1 year ago

    A merge request isn’t a git thing; It is an extension of some platforms. If you continue developing on the same branch as the mergerequest was issued for, github and gitlab will list your commits within the Merge Request once you have pushed them to your remote.

    If you do not want this, you can simply branch off after issuing your merge request.

    • larida@lemmy.sdf.orgOP
      link
      fedilink
      arrow-up
      1
      ·
      1 year ago

      ok, so a “merge request” can be viewed as “merge of branch1 into master will happen at a time I cannot control”.

      Now, branch1 is checked out, if I do git switch -c branch2, it will start a new branch2 based on the last commit from branch1, right? I feel it’s safer, since I don’t know when branch1 will merge, server-side.

      • mryessir@lemmy.sdf.org
        link
        fedilink
        arrow-up
        3
        ·
        1 year ago

        Exactly.

        Depending on the upstream guidelines (check for a CONTRIBUTION file) you may open a MR with your initial development efforts. And reuse the branch until you have finished the feature. Then you request a review.

        Or you may first mention your branch on a issue and only create the merge request once the entire feature is developed.

        If you are developing another feature, use a dedicated branch.

        In any case, the author merging may elect only specific portions of your change.

        Also note that it is perfectly normal that a merge request will be open for months. So don’t be discouraged. There may already be people profiting from your change. You just don’t see it.

        • TechNom (nobody)
          link
          fedilink
          English
          arrow-up
          1
          ·
          1 year ago

          I think that you can add a WIP prefix to the MR title to prevent the maintainers from accidentally merging the branch prematurely.

      • GarytheSnail
        link
        fedilink
        arrow-up
        1
        ·
        1 year ago

        If you have zero idea when but you assume it will be merged at some point, I think you’ve got the right idea.

        Do you know the merge strategy of the remote? Is it fast forward only, merge commit, squash then merge commit?