[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