Skip to content

Draft: Add merge request to merge train#2552

Closed
p-varma wants to merge 3 commits into
python-gitlab:mainfrom
p-varma:main
Closed

Draft: Add merge request to merge train#2552
p-varma wants to merge 3 commits into
python-gitlab:mainfrom
p-varma:main

Conversation

@p-varma

@p-varma p-varma commented Apr 20, 2023

Copy link
Copy Markdown

MR is in Draft state
Yet to write the testcases.

Changes

Documentation and testing

Please consider whether this PR needs documentation and tests. This is not required, but highly appreciated:

@codecov

codecov Bot commented Apr 20, 2023 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (f310764) 96.21% compared to head (ecaa566) 96.07%.
Report is 149 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #2552      +/-   ##
==========================================
- Coverage   96.21%   96.07%   -0.14%     
==========================================
  Files          87       87              
  Lines        5700     5726      +26     
==========================================
+ Hits         5484     5501      +17     
- Misses        216      225       +9
Flag Coverage Δ
api_func_v4 82.37% <66.66%> (-0.08%) ⬇️
cli_func_v4 83.02% <66.66%> (-0.09%) ⬇️
unit 87.77% <66.66%> (-0.11%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
gitlab/v4/objects/projects.py 98.86% <100.00%> (+<0.01%) ⬆️
gitlab/v4/objects/merge_trains.py 74.28% <64.00%> (-25.72%) ⬇️

robinson96
robinson96 reviewed Aug 16, 2023
View reviewed changes

@robinson96 robinson96 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My team is already using this branch for our workflow, so excited to have this piece of the puzzle.

FWIW - at least in current versions of Gitlab, we are seeing issues with the merge train actually being merged in if the SHA is not specified as part of the request. May consider having the SHA be required at this layer so others are able to dodge the pain.

@github-actions

github-actions Bot commented Feb 13, 2024

Copy link
Copy Markdown

This PR was closed because it has been marked stale for 15 days with no activity. If this PR is still valid, please re-open.

@github-actions github-actions Bot closed this Feb 13, 2024
@nejch nejch mentioned this pull request Jul 12, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@robinson96 robinson96 robinson96 left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

need info stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@p-varma @robinson96 @JohnVillalovos @prashantv-cb