Feat/merge trains additional #2547#3381
Feat/merge trains additional #2547#3381isaac-philip wants to merge 12 commits intopython-gitlab:mainfrom
Conversation
…dd mr Signed-off-by: Isaac Philip <4974658+isaac-philip@users.noreply.github.com>
…dd mr - test data modifications Signed-off-by: Isaac Philip <4974658+isaac-philip@users.noreply.github.com>
…dd mr - minor changes Signed-off-by: Isaac Philip <4974658+isaac-philip@users.noreply.github.com>
…dd mr - lint formatted by black Signed-off-by: Isaac Philip <4974658+isaac-philip@users.noreply.github.com>
…dd mr - docs added Signed-off-by: Isaac Philip <4974658+isaac-philip@users.noreply.github.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3381 +/- ##
==========================================
+ Coverage 95.75% 95.77% +0.01%
==========================================
Files 100 100
Lines 6125 6152 +27
==========================================
+ Hits 5865 5892 +27
Misses 260 260
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
|
@JohnVillalovos kindly have a look for same set of changes with earlier MR rebased on latest. thanks. |
There was a problem hiding this comment.
Pull request overview
Adds support for GitLab’s merge train merge-request endpoints (MR status on merge train + adding an MR to a merge train) to python-gitlab, along with unit tests and docs, addressing issue #2547.
Changes:
- Extend
ProjectMergeTrainManagerto supportget()and add a nestedmerge_requestsmanager for merge train MR status/add-to-train endpoints. - Add unit tests covering merge train listing, MR status retrieval, and adding an MR to a merge train.
- Document the new merge train MR classes/managers and provide usage examples.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
gitlab/v4/objects/merge_trains.py |
Adds merge train MR REST object + manager; enables get() on merge trains. |
gitlab/v4/objects/projects.py |
Adjusts import for merge trains manager (lint-sensitive). |
tests/unit/objects/test_merge_trains.py |
Adds fixtures and tests for new merge train MR endpoints. |
docs/gl_objects/merge_trains.rst |
Documents new merge train MR objects/managers and examples. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Add _parent_ref_attr class attribute to RESTManager that allows nested managers
|
@JohnVillalovos appreciate the review, kindly check if it looks good now after review changes. any more suggestions also I would be happy to rectify sooner. thank you. |
closes #2547