[JENKINS-47938] Add Support for More Github Events

[jenkins-infra-bot]

The github-plugin currently supports PUSH events from Github only:

github-plugin/src/main/java/org/jenkinsci/plugins/github/webhook/subscriber/DefaultPushGHEventSubscriber.java

Lines 54 to 60 in 78167fd

	/**
	* @return set with only push event
	*/
	@Override
	protected Set<GHEvent> events() {
	return immutableEnumSet(PUSH);
	}

We'd like to be able to take action based on other events such as a release/tag being cut (X-GitHub-Event: create). We poked around a bit and see that there is an open PR to support additional events (#54 Are there any plans to address the outstanding issues in the PR or add this feature, in general?

---

Originally reported by ryanfrantz, imported from: Add Support for More Github Events

• assignee: lanwen
• status: Open
• priority: Minor
• component(s): github-plugin
• resolution: Unresolved
• votes: 0
• watchers: 2
• imported: 2025-12-08

Raw content of original issue

The github-plugin currently supports PUSH events from Github only:


  

    

      github-plugin/src/main/java/org/jenkinsci/plugins/github/webhook/subscriber/DefaultPushGHEventSubscriber.java
    
    

        Lines 54 to 60
      in
      78167fd
    
  
  

    


        

          

          
     /** 
        

        

          

          
      * @return set with only push event 
        

        

          

          
      */ 
        

        

          

          
     @Override 
        

        

          

          
     protected Set<GHEvent> events() { 
        

        

          

          
         return immutableEnumSet(PUSH); 
        

        

          

          
     } 
        
    
  



We'd like to be able to take action based on other events such as a release/tag being cut (X-GitHub-Event: create). We poked around a bit and see that there is an open PR to support additional events (#54 Are there any plans to address the outstanding issues in the PR or add this feature, in general?

[jenkins-infra-bot]

lanwen:

• Original comment link
• Raw content of original comment:

  This change requires massive rewrite of plugin, as it originally was not designed for variable events. But for now I have no time to do such great things as it requires to put a lot of efforts doing this in java7-way with lot of backward-compatibility steps to not break any existing installation after the upgrade. With my current focus on avoiding jenkins it exists in the tail of very long backlog. So if you can implement it as new trigger probably with java7 I can help with review and merge with pleasure. PR#54 breaks some binary compatibility and generally contains a lot of issues because of tightly-coupled to a specific case.
  

This change requires massive rewrite of plugin, as it originally was not designed for variable events. But for now I have no time to do such great things as it requires to put a lot of efforts doing this in java7-way with lot of backward-compatibility steps to not break any existing installation after the upgrade. With my current focus on avoiding jenkins it exists in the tail of very long backlog. So if you can implement it as new trigger probably with java7 I can help with review and merge with pleasure. PR#54 breaks some binary compatibility and generally contains a lot of issues because of tightly-coupled to a specific case.