www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.learn - How to deal with interdependent dlang PRs?

reply Quirin Schroll <qs.il.paperinik gmail.com> writes:
I have 2 PRs, [one on 
dlang/dlang.org](https://github.com/dlang/dlang.org/pull/3446) 
and [one on dlang/dmd](https://github.com/dlang/dmd/pull/15245). 
The latter fails a test because an example on the (current) 
dlang.org fails. The dlang.org PR changes the example, and fails 
likewise it’s based on the changes in the dmd PR.

One measure I could think of:
1. Create a new dlang.org PR that (temporarily) removes the 
offending example; that one should not fail tests.
2. When it has been merged, the dmd PR will not fail anymore and 
can be merged.
3. The original dlang.org PR "reintroduces" the example with 
appropriate changes. It won’t fail because the compiler has been 
changed.

This seems a little convoluted. Is there a process? I can’t be 
the first one running into this.
May 25 2023
parent reply Dennis <dkorpel gmail.com> writes:
On Thursday, 25 May 2023 at 15:37:00 UTC, Quirin Schroll wrote:
 Is there a process? I can’t be the first one running into this.
Doing it in 3 PRs is the process. This is one of the reasons why druntime was merged into dmd's repository. I remember someone saying that if you name the git branches the same, the CI checks out the PR's corresponding branch in other repositories, but I have no experience doing this so I'm not sure it will work.
May 25 2023
parent Quirin Schroll <qs.il.paperinik gmail.com> writes:
On Thursday, 25 May 2023 at 20:18:08 UTC, Dennis wrote:
 On Thursday, 25 May 2023 at 15:37:00 UTC, Quirin Schroll wrote:
 Is there a process? I can’t be the first one running into this.
Doing it in 3 PRs is the process.
Okay. It’s not that bad.
 This is one of the reasons why druntime was merged into dmd's 
 repository. I remember someone saying that if you name the git 
 branches the same, the CI checks out the PR's corresponding 
 branch in other repositories, but I have no experience doing 
 this so I'm not sure it will work.
I’ll try that. I wonder what happens when I rename the branches on my fork, but renaming branches isn’t that uncommon, I guess.
May 26 2023