Skip to content

feat: Allow for any kwarg to be passed to python-gitlab by GitlabWrapper#1140

Merged
amimas merged 4 commits intogitlabform:mainfrom
thomascizeron:feat/allow-any-kwargs-to-python-gitlab
Dec 21, 2025
Merged

feat: Allow for any kwarg to be passed to python-gitlab by GitlabWrapper#1140
amimas merged 4 commits intogitlabform:mainfrom
thomascizeron:feat/allow-any-kwargs-to-python-gitlab

Conversation

@thomascizeron
Copy link
Copy Markdown
Contributor

Solves #1025

This PR enables GitlabForm to pass any additional configuration parameters from the gitlab section directly to the underlying python-gitlab Gitlab client. This provides users with flexibility to configure advanced python-gitlab features without requiring GitLabForm updates.

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.29%. Comparing base (738da13) to head (3778048).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1140      +/-   ##
==========================================
+ Coverage   87.17%   88.29%   +1.11%     
==========================================
  Files          73       73              
  Lines        3463     3461       -2     
==========================================
+ Hits         3019     3056      +37     
+ Misses        444      405      -39     
Flag Coverage Δ
integration 86.88% <100.00%> (+1.63%) ⬆️
unittests 37.82% <30.76%> (+0.19%) ⬆️
Files with missing lines Coverage Δ
gitlabform/gitlab/__init__.py 100.00% <100.00%> (ø)
gitlabform/gitlab/core.py 89.13% <100.00%> (-0.08%) ⬇️

... and 7 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Collaborator

@amimas amimas left a comment

Choose a reason for hiding this comment

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

Thank you so much for looking into this and the PR. Like the extra tests being included. Left some comments. Please let me know if you have any questions.

Comment thread docs/reference/index.md Outdated
Comment thread gitlabform/gitlab/core.py
Comment thread docs/reference/index.md Outdated
Comment thread docs/reference/index.md Outdated
Comment thread docs/reference/index.md Outdated
@thomascizeron thomascizeron force-pushed the feat/allow-any-kwargs-to-python-gitlab branch from c6ef449 to 3778048 Compare December 18, 2025 19:31
@thomascizeron thomascizeron temporarily deployed to Integrate Pull Request December 18, 2025 19:31 — with GitHub Actions Inactive
@thomascizeron thomascizeron temporarily deployed to Integrate Pull Request December 18, 2025 19:31 — with GitHub Actions Inactive
Copy link
Copy Markdown
Collaborator

@amimas amimas left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks again for working on this.

@amimas amimas merged commit 6802bd2 into gitlabform:main Dec 21, 2025
20 checks passed
amimas pushed a commit that referenced this pull request Jan 4, 2026
In python-gitlab client, arbitrary parameters to graphql client is not accepted gracefully. This change fixes this issue by inspecting the library's signature and passing valid parameters to it. This way, we still maintain ability to pass accepted parameters/configurations through gitlabform's config.

Related to #1140
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants