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

Option to use author instead of commiter in declarative pipeline

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      Problem
      Right now when adding a GitHub source in Blue Ocean every change merged from a pull request is being shown as "Changes from noreply". This also impacts all plugins that rely on the commiters for a build since it effectively hides the actual authors (including when using `email-ext` or `mailer`).

      Since changing the default behaviour might be a breaking change, an option should be added to the pipeline so that each agent uses the correct strategy for the checkout.

      (in `git-plugin` this is called `AuthorInChangelog`)

      Scope

      • Switch to displaying the Author information for Git repositories (instead of Committer)

        Attachments

          Issue Links

            Activity

            Hide
            rochdev Roch Devost added a comment -

            I am actually wondering if this should be reported as a bug instead, since the GitHub source is basically incompatible with GitHub.

            Show
            rochdev Roch Devost added a comment - I am actually wondering if this should be reported as a bug instead, since the GitHub source is basically incompatible with GitHub.
            Hide
            jamesdumay James Dumay added a comment -

            I don't think that the core API for change sets can differentiate between committer and author. I'd strongly suggest either it is updated to support that or we switch from one to another.

            Show
            jamesdumay James Dumay added a comment - I don't think that the core API for change sets can differentiate between committer and author. I'd strongly suggest either it is updated to support that or we switch from one to another.
            Hide
            abayer Andrew Bayer added a comment -

            As Roch Devost pointed out, the git plugin has this support already - https://github.com/jenkinsci/git-plugin/blob/master/src/main/java/hudson/plugins/git/extensions/impl/AuthorInChangelog.java - so I think this would be best as an option for the GitHub branch source, probably for BitBucket branch source as well.

            Show
            abayer Andrew Bayer added a comment - As Roch Devost pointed out, the git plugin has this support already - https://github.com/jenkinsci/git-plugin/blob/master/src/main/java/hudson/plugins/git/extensions/impl/AuthorInChangelog.java - so I think this would be best as an option for the GitHub branch source, probably for BitBucket branch source as well.
            Hide
            rochdev Roch Devost added a comment -

            If it is possible to enable it by default it would be best, since the current behaviour doesn't make much sense anyway. I can't think of a reason why anybody would want to keep `nobody` as the author of every commits.

            Show
            rochdev Roch Devost added a comment - If it is possible to enable it by default it would be best, since the current behaviour doesn't make much sense anyway. I can't think of a reason why anybody would want to keep `nobody` as the author of every commits.
            Hide
            stephenconnolly Stephen Connolly added a comment -

            JENKINS-43507 enabled the GitSCMExtension that provides AuthorInChangeLog

            Show
            stephenconnolly Stephen Connolly added a comment - JENKINS-43507 enabled the GitSCMExtension that provides AuthorInChangeLog
            Hide
            abayer Andrew Bayer added a comment -

            Seems like this may make sense to add as an option for Declarative - might actually end up doing it in git-plugin, though. We'll see.

            Show
            abayer Andrew Bayer added a comment - Seems like this may make sense to add as an option for Declarative - might actually end up doing it in git-plugin , though. We'll see.

              People

              • Assignee:
                abayer Andrew Bayer
                Reporter:
                rochdev Roch Devost
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated: