-
Notifications
You must be signed in to change notification settings - Fork 437
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[api] implement package fork command #14830
Conversation
Review app will appear here: http://obs-reviewlab.opensuse.org/adrianschroeter-fork_command |
ba424f8
to
4c7d0e2
Compare
@eduardoj btw, I failed to find a way to define parameters optional or required in the docu, is there a way? |
By default they are optional. To mark them as required, use: If you made some change in your development environment and didn't notice any change after reloading the page, please read: https://github.com/openSUSE/open-build-service/wiki/New-Swagger-API-doc#how-to-test-it For reference: https://swagger.io/docs/specification/describing-parameters/#required-optional Thank you for your effort on documenting this! |
78bb918
to
60570ff
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #14830 +/- ##
=======================================
Coverage 86.04% 86.05%
=======================================
Files 793 793
Lines 25963 25982 +19
=======================================
+ Hits 22340 22358 +18
- Misses 3623 3624 +1 |
On Donnerstag, 24. August 2023, 13:12:42 CEST Henne Vogelsang wrote:
@hennevogel commented on this pull request.
> @@ -712,6 +712,22 @@ Parameters:
XmlResult: status
+POST /source/<project>/<package>?cmd=fork&scmsync=url
I'm not sure I'd like to introduce another term for this. Can't we just make this an optional parameter for branch?
this term is known from git world and it is intenionally not "branch"
as it will not create a branch in the backend.
It is an independ instantiation without any auto-merging mechanics
from OBS side.
The branch tracking and merging must happen on git side.
…--
Adrian Schroeter ***@***.***>
Build Infrastructure Project Manager
SUSE Software Solutions Germany GmbH, Frankenstraße 146, 90461 Nürnberg, Germany
(HRB 36809, AG Nürnberg) Geschäftsführer: Ivo Totev
|
e0b4868
to
d29b18b
Compare
5831131
to
34df36e
Compare
47882f8
to
45a094e
Compare
given that Eduardo and Henne have also work on it, I assume this is fine now and will merge it next week. (ignoring the random test suite failures which are unrelated). |
This can be used to create a test build setup for a scmsync package based on an existing project. The base package may or may not exist. May or may not use scmsync. This is mainly implemented inside of BranchPackage class intentionaly, even when this is no source branch technically. But many api parts like repository setup can be shared this way. Co-Author: Henne Vogelsang <hvogel@opensuse.org>
@adrianschroeter sorry, this fell through my cracks... |
This got lost in openSUSE#14830
Fixes openSUSE#15770 This got lost somehow in openSUSE#14830
Fixes openSUSE#15770 This got lost somehow in openSUSE#14830
In case the target package already exists for some reason - this is an updated pull request - the target package was created previously we still need to update the scmsync attribute. This broke with openSUSE#14830
In case the target package already exists for some reason - this is an updated pull request - the target package was created previously we still need to update the scmsync attribute. This broke with openSUSE#14830
This can be used to create a test build setup for a scmsync package based on an existing project.
The base package may or may not exist. May or may not use scmsync.
This is mainly implemented inside of BranchPackage class intentionaly, even when this is no source branch technically. But many api parts like repository setup can be shared this way.