-
Notifications
You must be signed in to change notification settings - Fork 621
Add subgroup event support and update pom.xml #1833
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
|
Hi @n0kke may I ask what is this PR intended for? |
The purpose of this PR is to add support for GitLab subgroup webhook events, which are not currently handled by the plugin (along with other event types). This change enables Jenkins jobs to respond correctly to events originating from GitLab subgroups, aligning their behaviour with that of group and project events. The update to the pom.xml was necessary solely to resolve build and dependency compatibility issues encountered during the implementation and testing of this change, and does not affect the plugin's runtime behaviour. |
|
|
||
| <artifactId>gitlab-plugin</artifactId> | ||
| <version>${revision}${changelist}</version> | ||
| <version>1.9.10-SNAPSHOT</version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| <version>1.9.10-SNAPSHOT</version> | |
| <version>${revision}${changelist}</version> |
Please undo this change
| <plugin> | ||
| <artifactId>maven-enforcer-plugin</artifactId> | ||
| <executions> | ||
| <execution> | ||
| <id>display-info</id> | ||
| <configuration> | ||
| <rules> | ||
| <requireUpperBoundDeps> | ||
| <excludes combine.children="append"> | ||
| <exclude>com.sun.activation:jakarta.activation</exclude> | ||
| <exclude>jakarta.xml.bind:jakarta.xml.bind-api</exclude> | ||
| </excludes> | ||
| </requireUpperBoundDeps> | ||
| </rules> | ||
| </configuration> | ||
| </execution> | ||
| </executions> | ||
| </plugin> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this needed?
Testing done
Submitter checklist