How to deal with interdependent dlang PRs?

2023-05-25 Thread Quirin Schroll via Digitalmars-d-learn
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.


Re: How to deal with interdependent dlang PRs?

2023-05-25 Thread Dennis via Digitalmars-d-learn

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.




Re: How to deal with interdependent dlang PRs?

2023-05-26 Thread Quirin Schroll via Digitalmars-d-learn

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.