• Icon: New Feature New Feature
    • Resolution: Fixed
    • Icon: Minor Minor
    • scm-api-plugin
    • None

      Post JENKINS-43507 with SCM-API having non-trivial implementations of instantiable things, we should add @Symbol annotations to the relevant descriptors.

          [JENKINS-44884] Add @Symbol annotations

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          pom.xml
          src/main/java/jenkins/scm/impl/SingleSCMNavigator.java
          src/main/java/jenkins/scm/impl/SingleSCMSource.java
          src/main/java/jenkins/scm/impl/trait/RegexSCMHeadFilterTrait.java
          src/main/java/jenkins/scm/impl/trait/RegexSCMSourceFilterTrait.java
          src/main/java/jenkins/scm/impl/trait/WildcardSCMHeadFilterTrait.java
          src/main/java/jenkins/scm/impl/trait/WildcardSCMSourceFilterTrait.java
          src/test/java/jenkins/scm/impl/SymbolAnnotationsTest.java
          src/test/java/jenkins/scm/impl/mock/MockSCM.java
          src/test/java/jenkins/scm/impl/mock/MockSCMDiscoverBranches.java
          src/test/java/jenkins/scm/impl/mock/MockSCMDiscoverChangeRequests.java
          src/test/java/jenkins/scm/impl/mock/MockSCMDiscoverTags.java
          src/test/java/jenkins/scm/impl/mock/MockSCMNavigator.java
          src/test/java/jenkins/scm/impl/mock/MockSCMSource.java
          http://jenkins-ci.org/commit/scm-api-plugin/d16e1eb8afcb159881fdc1a797e7d2d44e595bd4
          Log:
          [FIXED JENKINS-44884] Add @Symbol annotations

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: pom.xml src/main/java/jenkins/scm/impl/SingleSCMNavigator.java src/main/java/jenkins/scm/impl/SingleSCMSource.java src/main/java/jenkins/scm/impl/trait/RegexSCMHeadFilterTrait.java src/main/java/jenkins/scm/impl/trait/RegexSCMSourceFilterTrait.java src/main/java/jenkins/scm/impl/trait/WildcardSCMHeadFilterTrait.java src/main/java/jenkins/scm/impl/trait/WildcardSCMSourceFilterTrait.java src/test/java/jenkins/scm/impl/SymbolAnnotationsTest.java src/test/java/jenkins/scm/impl/mock/MockSCM.java src/test/java/jenkins/scm/impl/mock/MockSCMDiscoverBranches.java src/test/java/jenkins/scm/impl/mock/MockSCMDiscoverChangeRequests.java src/test/java/jenkins/scm/impl/mock/MockSCMDiscoverTags.java src/test/java/jenkins/scm/impl/mock/MockSCMNavigator.java src/test/java/jenkins/scm/impl/mock/MockSCMSource.java http://jenkins-ci.org/commit/scm-api-plugin/d16e1eb8afcb159881fdc1a797e7d2d44e595bd4 Log: [FIXED JENKINS-44884] Add @Symbol annotations

          James Dumay added a comment -

          stephenconnolly looks like its been merged. Ready to resolve this one?

          James Dumay added a comment - stephenconnolly looks like its been merged. Ready to resolve this one?

          Flagging the follow-up work

          Stephen Connolly added a comment - Flagging the follow-up work

          The SCM-API traits have @Symbol annotations and we have a follow-up issue for the other traits

          Stephen Connolly added a comment - The SCM-API traits have @Symbol annotations and we have a follow-up issue for the other traits

            stephenconnolly Stephen Connolly
            stephenconnolly Stephen Connolly
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: