Codecov Report

Attention: Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 91.12%. Comparing base (e54832a) to head (1730fcb).
Report is 1411 commits behind head on main.

@@            Coverage Diff             @@
##             main    #1577      +/-   ##
==========================================
- Coverage   91.16%   91.12%   -0.04%     
==========================================
  Files          74       74              
  Lines        4177     4182       +5     
==========================================
+ Hits         3808     3811       +3     
- Misses        369      371       +2     

Hi @GerevSec, thanks a lot for your contribution! We just need a few things to get this green:

1. git commit --amend your commit message to follow conventional commit, e.g. feat(objects): add merge_base API support
2. run black to format code properly, e.g. with tox -e black
3. If possible, could you add some documentation and unit tests? for inspiration, see:
   • docs: add an example, like in https://github.com/python-gitlab/python-gitlab/blob/master/docs/gl_objects/projects.rst
   • tests: add a small unit test here https://github.com/python-gitlab/python-gitlab/blob/master/tests/unit/objects/test_repositories.py

This Pull Request (PR) was marked stale because it has been open 90 days with no activity. Please remove the stale label or comment on this PR. Otherwise, it will be closed in 15 days.

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.

@nejch @GerevSec
Hi.
Do you have any plan to develop this feature?

Hi @nejch
Would you want me to continue this Pull Request? I think this feature will be useful.

Thanks for the ping again @emadimehrdad, this one slipped a little bit. Yes, we'll pick it up and add some tests to get this merged :)

As it is, I don't think we can merge a PR with this change alone - the API expects 2 or more refs (arbitrary number), which need to be supplied as an array. So we'll also need #1699 first.