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

Naming inconsistency between GitHub and Bitbucket

XMLWordPrintable

    • 43.vea_da_7c53b_831

      Hello,

      I'd like to enhance the docs of this plugin and create a PR for that.

      However, I've realized that there is a naming inconsistency between GitHub and Bitbucket which is super confusing.

      Not sure if that intended or there are any constraints that force that naming inconsistency.

      Thanks

          [JENKINS-64360] Naming inconsistency between GitHub and Bitbucket

          Ahmed AbouZaid created issue -
          Ahmed AbouZaid made changes -
          Component/s New: skip-notifications-trait-plugin [ 23947 ]
          Component/s Original: atlassian-bitbucket-server-integration-plugin [ 25721 ]
          Ahmed AbouZaid made changes -
          Issue Type Original: Bug [ 1 ] New: Improvement [ 4 ]

          Jack Whelpton added a comment - - edited

          I think the naming has to be different to avoid clashes (see, e.g. https://issues.jenkins.io/browse/JENKINS-61464). That said, it feels like clearer naming might be:

          skipBitbucketNotifications
          skipGitHubNotifications
          skipGitLabNotifications

          etc. It's more verbose, but seems more consistent and extensible than the current approach.

          Jack Whelpton added a comment - - edited I think the naming has to be different to avoid clashes (see, e.g. https://issues.jenkins.io/browse/JENKINS-61464 ). That said, it feels like clearer naming might be: skipBitbucketNotifications skipGitHubNotifications skipGitLabNotifications etc. It's more verbose, but seems more consistent and extensible than the current approach.

          Ahmed AbouZaid added a comment -

          jackwhelpton I really like your suggestion

           

          Ahmed AbouZaid added a comment - jackwhelpton I really like your suggestion  

          Valentin Delaye added a comment - - edited

          Hi,

          In gitlab plugin the symbol is named 'gitlabSkipNotifications' : https://github.com/jenkinsci/gitlab-branch-source-plugin/pull/79

          I'm planning to rename the symbols (and classes)

          bitbucketSkipNotifications
          githubSkipNotifications

          Regards,

          Valentin Delaye added a comment - - edited Hi, In gitlab plugin the symbol is named 'gitlabSkipNotifications' : https://github.com/jenkinsci/gitlab-branch-source-plugin/pull/79 I'm planning to rename the symbols (and classes) bitbucketSkipNotifications githubSkipNotifications Regards,
          Valentin Delaye made changes -
          Assignee Original: Kristy Hughes [ khughes ] New: Valentin Delaye [ jonesbusy ]
          Valentin Delaye made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Valentin Delaye made changes -
          Status Original: In Progress [ 3 ] New: Open [ 1 ]
          Valentin Delaye made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Fixed but Unreleased [ 10203 ]

            jonesbusy Valentin Delaye
            aabouzaid Ahmed AbouZaid
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: