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

Exceptions in generated DSL documentation for `unarchive`

    Details

    • Similar Issues:

      Description

      See attached screenshot

        Attachments

          Issue Links

            Activity

            Hide
            rtyler R. Tyler Croy added a comment -

            FWIW this is still present as of Pipeline 2.3 / Jenkins 2.7.3

            Show
            rtyler R. Tyler Croy added a comment - FWIW this is still present as of Pipeline 2.3 / Jenkins 2.7.3
            Hide
            abayer Andrew Bayer added a comment -

            Jesse Glick - so I'm taking a look at this and I'm not sure what the right approach is. Do we need a MapType a la ArrayType?

            Show
            abayer Andrew Bayer added a comment - Jesse Glick - so I'm taking a look at this and I'm not sure what the right approach is. Do we need a MapType a la ArrayType ?
            Hide
            jglick Jesse Glick added a comment -

            No, it is a waste of time since ArtifactUnarchiverStep/config.jelly does not even try to support visual configuration. Should just move that blurb into help.html and make the f:block say to look at it (so that the static reference shows docs for the step); and make html.groovy quietly skip over the mappings attribute somehow, without showing the ugly exception.

            Show
            jglick Jesse Glick added a comment - No, it is a waste of time since ArtifactUnarchiverStep/config.jelly does not even try to support visual configuration. Should just move that blurb into help.html and make the f:block say to look at it (so that the static reference shows docs for the step); and make html.groovy quietly skip over the mappings attribute somehow, without showing the ugly exception.

              People

              • Assignee:
                Unassigned
                Reporter:
                rtyler R. Tyler Croy
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated: