Edit Group Import Annotations
This edits the file
type in the documentation only as it is an edge case where API::Validations::Types::WorkhorseFile
is not understood by grape-swagger
and will be excluded from auto-generation. By using the documentation
object, this only affects the generated OpenAPI documentation.
The OpenAPI V2 documentation will be used for dogfooding API fuzzing as per #372422 (closed) and the OpenAPI autogeneration &8926.
Merge request reports
Activity
changed milestone to %15.6
added Importer:Group Export/Import api backend + 1 deleted label
requested review from @Andysoiron
added devopsmanage sectiondev labels
Hey @elim-ext!
👋 Thank you for your contribution to GitLab. Please refer to the contribution flow documentation for a quick overview of the process, and the merge request (MR) guidelines for the detailed process.
When you're ready for a first review, post
@gitlab-bot ready
. If you know a relevant reviewer(s) (for example, someone that was involved in a related issue), you can also assign them directly with@gitlab-bot ready @user1 @user2
.At any time, if you need help moving the MR forward, feel free to post
@gitlab-bot help
. Read more on how to get help.To enable automated checks on your MR, please configure Danger for your fork.
You can comment
@gitlab-bot label <label1> <label2>
to add labels to your MR. Please see the list of allowed labels in thelabel
command documentation.This message was generated automatically. You're welcome to improve it.
added Community contribution workflowin dev labels
1 Warning ⚠ featureaddition and featureenhancement merge requests normally have a documentation change. Consider adding a documentation update or confirming the documentation plan with the Technical Writer counterpart.
For more information, see:
- The Handbook page on merge request types.
- The definition of done documentation.
1 Message 📖 CHANGELOG missing: If you want to create a changelog entry for GitLab FOSS, add the
Changelog
trailer to the commit message you want to add to the changelog.If you want to create a changelog entry for GitLab EE, also add the
EE: true
trailer to your commit message.If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.
Reviewer roulette
Changes that require review have been detected!
Please refer to the table below for assigning reviewers and maintainers suggested by Danger in the specified category:
Category Reviewer Maintainer backend Michael Becker ( @wandering_person
) (UTC-7)Etienne Baqué ( @ebaque
) (UTC+4)To spread load more evenly across eligible reviewers, Danger has picked a candidate for each review slot, based on their timezone. Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.
To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.
Once you've decided who will review this merge request, assign them as a reviewer! Danger does not automatically notify them for you.
If needed, you can retry the
🔁 danger-review
job that generated this comment.Generated by
🚫 DangerEdited by Ghost Usermentioned in issue gitlab-org/quality/triage-reports#9573 (closed)
added featureenhancement label
added typefeature label
- Resolved by Aleksei Lipniagov
Thank you @elim-ext! Looks good
👍 @alipniagov can you do the maintainer review?
requested review from @alipniagov and removed review request for @Andysoiron
- Resolved by Aleksei Lipniagov
👋 @Andysoiron
, thanks for approving this merge request.This is the first time the merge request is approved. To ensure full test coverage, please start a new pipeline before merging.
For more info, please refer to the following links:
added documentation label
enabled an automatic merge when the pipeline for 6f3a89ca succeeds