[all-commits] [llvm/llvm-project] dfc406: [llvm][docs] Clean up the "Landing Your Change" se...

[David Spickett via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: dfc40650eccef9ec9a317e7cfe65d9d4fc8fe618
      https://github.com/llvm/llvm-project/commit/dfc40650eccef9ec9a317e7cfe65d9d4fc8fe618
  Author: David Spickett <david.spickett at linaro.org>
  Date:   2024-10-23 (Wed, 23 Oct 2024)

  Changed paths:
    M llvm/docs/GitHub.rst

  Log Message:
  -----------
  [llvm][docs] Clean up the "Landing Your Change" section of the GitHub docs (#112869)

* Note up front that the author may not have permissions to use the
merge button and should ask a reviewer to do those steps.
* Make it clear that a single commit PR can be landed with a single
button click.
* There are in fact 3 ways to land a multi-commit PR.
* Order the ways in increasing amount of overhead for the PR author.
* Put them in bullet point sections so they are visually separate.
* Add a note that force pushes can be problematic when the PR has multiple authors, but don't go too much into how to solve that, Git's docs are better here anyway.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications