[libc-commits] [libc] [libc] Add `dlfcn.h` placeholder (PR #97501)
Izaak Schroeder via libc-commits
libc-commits at lists.llvm.org
Sat Jul 6 13:00:12 PDT 2024
izaakschroeder wrote:
> There weren't any merge conflicts, were there? If you amend and force-push to have only 1 commit on the pull request, then you shouldn't do that. See https://llvm.org/docs/GitHub.html#updating-pull-requests.
>
> Even if there are merge conflicts, you should rebase your existing commits instead of resetting your branch and creating a single new commit.
Ok I can do that no problem 😀 I do like the idea of tracking review changes... though my habit has normally been to clean everything up before merging and I guess I had figured we were pretty close. Thanks for the heads up!
https://github.com/llvm/llvm-project/pull/97501
More information about the libc-commits
mailing list