Want to submit a PR while another one is awaiting approval... #3782
Replies: 2 comments 1 reply
-
|
Thanks for your efforts! Feel free to create a new PR from your newer changes. You can leave #3660 open, convert it to draft or close it. This won't make it disappear.
You can fork Jamulus into your own repo and work on as many branches as you'd like to add. You can then create PRs from every branch you created. No limits ;) My workflow is this:
|
Beta Was this translation helpful? Give feedback.
-
|
jamulusclient/connect and disconnect please put this in a separate PR. It is quite hard to get correct as the GUI must remain in sync. Last time someone tried it it wasn't approvable as the UI went out of sync. It's still a very important feature. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
I have a PR, Add new JSON-RPC API calls, with documentation #3660 (comment), that has not yet been approved (generated a bunch of discussion) - related to new JSON-RPC API calls.
I have a new PR to submit.
a. This is an extension of the current pending PR that adds more API calls, and actually copies the ones previously submitted.
b. New API calls:
1'. jamulusclient/connect
2'. jamulusclient/disconnect
3'. jamulusclient/setCurrentDirectory
c. New attribute to jamulusclient/getChannelInfo: result.id // The channel ID assigned by the server, or -1 if not currently connected.
d. This PR will need review attention - there's re-factoring in order to extract the common connect and disconnect logic used by both the UI and these new API calls. Note: Calling the API calls updates the UI (and states, such as the connect/disconnect buttons). This has been tested, but not to the extent I feel completely comfortable with (let's say there's a modicum of confidence). And I still get to play my newbie card.
Github will only let me own one fork of a repo - which is the one currently used for the pending PR.
a. I could withdraw the current PR, freeing up my 1-fork allowance - but concerned about losing the valuable discussion that occurred as well as any review work that has been already done.
b. I could create another temporary username/account, which opens up a fork slot (would rather not do that).
c. I could wait for the PR to get approved, so my current fork is no longer needed, and I can overwrite with this new submission.
Btw: I took the comment of wanting the API to have parity with the UI to heart - which was the genesis of these new API calls in order to get a bit closer to that comment. This is certainly not all of the gaps (I have a list that Claude found, which is for a separate conversation).
How should I proceed?
Beta Was this translation helpful? Give feedback.
All reactions