Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-27154

Make the github-plugin commit status configurable instead of tied to the project name

    Details

    • Similar Issues:

      Description

      Allow configuration of the commit status name instead of only using the full job name. This should take into account environment variables to better support matrix builds as well.

        Attachments

          Issue Links

            Activity

            Hide
            s0undt3ch Pedro Algarvio added a comment -

            This issue was cloned from https://issues.jenkins-ci.org/browse/JENKINS-23995

            The difference is the *configurable* commit status context.

            Either a simple configuration variable per job config, or to go the extra mile, make this variable access the build environment variables.

            Show
            s0undt3ch Pedro Algarvio added a comment - This issue was cloned from https://issues.jenkins-ci.org/browse/JENKINS-23995 The difference is the * configurable * commit status context. Either a simple configuration variable per job config, or to go the extra mile, make this variable access the build environment variables.
            Hide
            urkle Edward Rudd added a comment -

            I've cleaned up the summary and description of the issue to reflect the new request.

            Show
            urkle Edward Rudd added a comment - I've cleaned up the summary and description of the issue to reflect the new request.
            Hide
            s0undt3ch Pedro Algarvio added a comment -

            Thanks!

            Show
            s0undt3ch Pedro Algarvio added a comment - Thanks!
            Hide
            lanwen Kirill Merkushev added a comment -

            Good point. Will think about it next

            Show
            lanwen Kirill Merkushev added a comment - Good point. Will think about it next
            Hide
            integer Kanstantsin Shautsou added a comment -

            I think universal message calculation/ui configuration can be migrated from github-pullrequest-plugin, but default values for this Describables under question.

            Show
            integer Kanstantsin Shautsou added a comment - I think universal message calculation/ui configuration can be migrated from github-pullrequest-plugin, but default values for this Describables under question.
            Hide
            lanwen Kirill Merkushev added a comment -
            Show
            lanwen Kirill Merkushev added a comment - Working on it here: https://github.com/jenkinsci/github-plugin/pull/100
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Merkushev Kirill
            Path:
            pom.xml
            src/main/java/com/cloudbees/jenkins/GitHubCommitNotifier.java
            src/main/java/com/cloudbees/jenkins/GitHubSetCommitStatusBuilder.java
            src/main/java/com/coravy/hudson/plugins/github/GithubProjectProperty.java
            src/main/java/org/jenkinsci/plugins/github/common/ExpandableMessage.java
            src/main/resources/com/cloudbees/jenkins/GitHubCommitNotifier/config.groovy
            src/main/resources/com/cloudbees/jenkins/GitHubCommitNotifier/config.jelly
            src/main/resources/com/cloudbees/jenkins/GitHubSetCommitStatusBuilder/config.groovy
            src/main/resources/com/cloudbees/jenkins/Messages.properties
            src/main/resources/com/coravy/hudson/plugins/github/GithubProjectProperty/config.groovy
            src/main/resources/com/coravy/hudson/plugins/github/GithubProjectProperty/config.jelly
            src/main/resources/com/coravy/hudson/plugins/github/GithubProjectProperty/config.properties
            src/main/resources/com/coravy/hudson/plugins/github/GithubProjectProperty/config_de.properties
            src/main/resources/com/coravy/hudson/plugins/github/GithubProjectProperty/help-displayName.html
            src/main/resources/org/jenkinsci/plugins/github/Messages.properties
            src/main/resources/org/jenkinsci/plugins/github/common/ExpandableMessage/config.groovy
            src/main/resources/org/jenkinsci/plugins/github/common/ExpandableMessage/help-content.html
            src/test/java/com/cloudbees/jenkins/GitHubCommitNotifierTest.java
            src/test/java/com/cloudbees/jenkins/GitHubSetCommitStatusBuilderTest.java
            src/test/java/org/jenkinsci/plugins/github/common/ExpandableMessageTest.java
            http://jenkins-ci.org/commit/github-plugin/8b9535d1292703a3db50e614e106235041b259b2
            Log:
            Merge pull request #100 from lanwen/expandable

            [FIXES JENKINS-27154] configurable build context and message in status builder and notifier

            Compare: https://github.com/jenkinsci/github-plugin/compare/1ed2eddc33d5...8b9535d12927

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Merkushev Kirill Path: pom.xml src/main/java/com/cloudbees/jenkins/GitHubCommitNotifier.java src/main/java/com/cloudbees/jenkins/GitHubSetCommitStatusBuilder.java src/main/java/com/coravy/hudson/plugins/github/GithubProjectProperty.java src/main/java/org/jenkinsci/plugins/github/common/ExpandableMessage.java src/main/resources/com/cloudbees/jenkins/GitHubCommitNotifier/config.groovy src/main/resources/com/cloudbees/jenkins/GitHubCommitNotifier/config.jelly src/main/resources/com/cloudbees/jenkins/GitHubSetCommitStatusBuilder/config.groovy src/main/resources/com/cloudbees/jenkins/Messages.properties src/main/resources/com/coravy/hudson/plugins/github/GithubProjectProperty/config.groovy src/main/resources/com/coravy/hudson/plugins/github/GithubProjectProperty/config.jelly src/main/resources/com/coravy/hudson/plugins/github/GithubProjectProperty/config.properties src/main/resources/com/coravy/hudson/plugins/github/GithubProjectProperty/config_de.properties src/main/resources/com/coravy/hudson/plugins/github/GithubProjectProperty/help-displayName.html src/main/resources/org/jenkinsci/plugins/github/Messages.properties src/main/resources/org/jenkinsci/plugins/github/common/ExpandableMessage/config.groovy src/main/resources/org/jenkinsci/plugins/github/common/ExpandableMessage/help-content.html src/test/java/com/cloudbees/jenkins/GitHubCommitNotifierTest.java src/test/java/com/cloudbees/jenkins/GitHubSetCommitStatusBuilderTest.java src/test/java/org/jenkinsci/plugins/github/common/ExpandableMessageTest.java http://jenkins-ci.org/commit/github-plugin/8b9535d1292703a3db50e614e106235041b259b2 Log: Merge pull request #100 from lanwen/expandable [FIXES JENKINS-27154] configurable build context and message in status builder and notifier Compare: https://github.com/jenkinsci/github-plugin/compare/1ed2eddc33d5...8b9535d12927
            Hide
            lanwen Kirill Merkushev added a comment -

            released as 1.14.1

            Show
            lanwen Kirill Merkushev added a comment - released as 1.14.1

              People

              • Assignee:
                lanwen Kirill Merkushev
                Reporter:
                s0undt3ch Pedro Algarvio
              • Votes:
                5 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: