[llvm-branch-commits] [libcxx] [libc++][modules] Adds module testing. (PR #76246)

[Mark de Wever via llvm-branch-commits]

mordante wrote:

> > > If you are okay with the suggestions that I made for some typos in the documentation, I will preemptively incorporate it into my documentation PR.
> > 
> > 
> > Thanks for the suggestions! Please do no incorporate these in your PR. Other reviewers may have other suggestions which you then need to incorporate too. It's a lot easier, for both of us, when you finish the documentation PR and after you merged your branch I move the comments of this patch to the new location. I left this comment for other reviewers so they don't need to comment on it. This is how we typically resolve conflicts between patches.
> 
> That makes total sense. I had already added the following to my PR (in case you are interested in incorporating here):
> 
> but I will gladly revert that if you think it's a good idea.

Yes otherwise we need to block your patch on this patch.

> Sorry for the confusion -- just trying to be helpful!

No problem, I appreciate your help!

https://github.com/llvm/llvm-project/pull/76246