-
Notifications
You must be signed in to change notification settings - Fork 14
Move to the @QubesOS organization #18
Comments
PR allows a convenient review workflow, but in case of repo like this, full rebase may be quite inconvenient. An alternative, I did in cases like this, is to do the full review by adding review comments in a separate commit (as normal comments in the code) - and have it as a PR. Then address those comments separately (separate PRs, or commits directly to the default branch) and rebase review commit from time to time, removing already addressed comments. Never actually merge that review PR. |
It is a pitty that this project seems to have stalled. |
Thanks for the reminder @runephilosof! I’m going to create the repo now. |
Repo created: https://github.com/QubesOS/qubes-video-companion. It (deliberately) has no commits yet so that whichever means we decide to go with will work. |
@marmarek decision on approach with regards to the new repo & PR? |
@marmarek is there a dead line for this? |
Glad to see @QubesOS get this upstreamed into the default Qubes installation! I'm chuffed to have helped in the start of bringing video sharing to the operating system! Thank you to everyone who's chipped in with development, testing, and support! This project was fun and I learned a lot making it. You can expect more contributions from me in the future (although probably not with quite the same magnitude as this project)! I'll be archiving the repo now... |
Qubes Video Companion will be a core part of a future Qubes OS release. This requires that it be moved to the @QubesOS organization on GitHub. Would creating a blank repository in @QubesOS and you creating a PR work?
The text was updated successfully, but these errors were encountered: