Skip to content

Missing push rules properties#1319

Merged
jmini merged 4 commits intogitlab4j:mainfrom
wilvdb:missing-push-rules
May 2, 2026
Merged

Missing push rules properties#1319
jmini merged 4 commits intogitlab4j:mainfrom
wilvdb:missing-push-rules

Conversation

@wilvdb
Copy link
Copy Markdown
Contributor

@wilvdb wilvdb commented Mar 26, 2026

We are using gitlab4j with Gilab 18.6 and it seems now, properties : commit_committer_name_check and reject_non_dco_commits are mandatory when creating or updating push rules.

Copy link
Copy Markdown
Collaborator

@jmini jmini 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 to me. I would like the corresponding example json file updated

.withParam("max_file_size", pushRule.getMaxFileSize())
.withParam("commit_committer_check", pushRule.getCommitCommitterCheck())
.withParam("reject_unsigned_commits", pushRule.getRejectUnsignedCommits());
.withParam("commit_committer_name_check", pushRule.getCommitCommitterNameChack())
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

typo

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Solved with edcec0c

Comment thread gitlab4j-api/src/main/java/org/gitlab4j/api/ProjectApi.java Outdated
Comment thread gitlab4j-models/src/main/java/org/gitlab4j/api/models/PushRules.java Outdated
@jmini jmini merged commit da12d98 into gitlab4j:main May 2, 2026
2 checks passed
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