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

Make Subversion Tag actions optional and transient

    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Minor Minor
    • subversion-plugin
    • None

      There's probably no need to pollute the build records with these, and they're easier to disable/remove that way if not needed.

          [JENKINS-24127] Make Subversion Tag actions optional and transient

          Mario Jauvin added a comment - - edited

          Can someone in the know ( like ifernandezcalvo) offer guidance.  I would be willing to make the change but

          • I don't know how to add UI options to a jenkins plugin (let alone the subversion plugin)
          • I don't have the knowledge on how to test.

          If someone can give pointers on how to do those two, I would be willing to submit a pull request.  I know that line 887 in SubversionSCM.java needs a conditional based on a UI option but I must confess that I have no idea how to implement that.

          Mario Jauvin added a comment - - edited Can someone in the know ( like ifernandezcalvo ) offer guidance.  I would be willing to make the change but I don't know how to add UI options to a jenkins plugin (let alone the subversion plugin) I don't have the knowledge on how to test. If someone can give pointers on how to do those two, I would be willing to submit a pull request.  I know that line 887 in SubversionSCM.java needs a conditional based on a UI option but I must confess that I have no idea how to implement that.

          Ivan Fernandez Calvo added a comment - For the UI you can check https://www.jenkins.io/doc/developer/forms/jelly-form-controls/ also on the SAML plugin you have an example of conditional sections in the UI https://github.com/jenkinsci/saml-plugin/blob/master/src/main/resources/org/jenkinsci/plugins/saml/SamlSecurityRealm/config.jelly For test you can check https://github.com/jenkinsci/saml-plugin/blob/master/src/test/java/org/jenkinsci/plugins/saml/SamlSecurityRealmTest.java Or https://github.com/jenkinsci/ssh-slaves-plugin/blob/master/src/test/java/hudson/plugins/sshslaves/SSHLauncherTest.java#L165 are my test about configuration in other test Also there is a sample plugin with a bunch of UI configurations https://github.com/jenkinsci/ui-samples-plugin

            Unassigned Unassigned
            danielbeck Daniel Beck
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: