-
-
Notifications
You must be signed in to change notification settings - Fork 5.9k
Fix the response format for actions/workflows
.
#35009
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also cc @bencurio and @ChristopherHX
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I missed this in the original PR
I think adding a Backport label for 1.24 makes sense to fix the openapi there as well
I was unable to create a backport for 1.24. @ScionOfDesign, please send one manually. 🍵
|
Hmm, swagger conflict makes it impossible to backport automatically. If you'd like to try to backport, you can checkout 1.24 branch and apply the change to it again (manually 🤣 ) and submit a new PR targeting 1.24 branch. If there is any difficulty, other maintainers and I could also help. |
Backport go-gitea#35009 This PR fixes the response format for the OpenAPI Spec of `ActionsListRepositoryWorkflows`. It was specified in the OpenAPI spec as returning a `[]*ActionWorkflow`, but it actually should return a `api.ActionWorkflowResponse`. The test already expects an `api.ActionWorkflowResponse` like expected.
We could improve the backport program here, e.g. if the only conflict is swagger delte the generated file, regenerate. I am doing this now manually |
Backport #35009 This PR fixes the response format for the OpenAPI Spec of `ActionsListRepositoryWorkflows`. It was specified in the OpenAPI spec as returning a `[]*ActionWorkflow`, but it actually should return a `api.ActionWorkflowResponse`. The test already expects an `api.ActionWorkflowResponse` like expected. Co-authored-by: Scion <Filiecs2@gmail.com>
* giteaofficial/main: [skip ci] Updated translations via Crowdin Docs/fix typo and grammar in CONTRIBUTING.md (go-gitea#35024) fix: improve english grammar and readability in locale_en-US.ini (go-gitea#35017) Add labeler config for `topic/code-linting` (go-gitea#35020) Fix various problems (go-gitea#35012) Fix the response format for `actions/workflows`. (go-gitea#35009)
This PR fixes the response format for the OpenAPI Spec of
ActionsListRepositoryWorkflows
.It was specified in the OpenAPI spec as returning a
[]*ActionWorkflow
, but it actually should return aapi.ActionWorkflowResponse
.The test already expects an
api.ActionWorkflowResponse
like expected.