FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add the ability to ignore push events from certain users by negasora · Pull Request #448 · jenkinsci/github-plugin · GitHub

Add the ability to ignore push events from certain users - #448

Open
negasora wants to merge 1 commit into
jenkinsci:masterfrom
negasora:add-ignored-users
Open

Add the ability to ignore push events from certain users#448
negasora wants to merge 1 commit into
jenkinsci:masterfrom
negasora:add-ignored-users

Conversation

negasora commented Oct 7, 2025
edited by KostyaSha
Loading

Copy link
Copy Markdown

This adds the ability to specify (per-job) usernames that should be ignored when determining whether or not to trigger a git poll from a GitHub webhook event.

This addresses JENKINS-48139 and JENKINS-24208

There have been a handful of PRs that have tried to fix this or similar issues (#58, #182, #83). They have either been closed or abandoned (afaict).

Testing done

Added testcases in GitHubPushTriggerTest.java and DefaultPushGHEventListenerTest.java

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests that demonstrate the feature works or the issue is fixed

This change is 

negasora force-pushed the add-ignored-users branch 2 times, most recently from 047f8ce to ed93ca7 Compare October 7, 2025 15:51
negasora marked this pull request as ready for review October 7, 2025 15:59
negasora requested a review from a team as a code owner October 7, 2025 15:59
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

1 participant


Back | FazBrowse Home | New Git URL