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

Provide option to disable token macro parsing in description files

      My markdown description files have many references to environment variables with backticks around them (as it explains the jenkins job scripts themselves, which use environment variables).

      While I'm sure I could ask devs to escape any comments they make, it's confusing to have a build fail for something in the meta-docs. It would be great if I could just disable token macro parsing at global- or job-level.

      Thanks!

          [JENKINS-14088] Provide option to disable token macro parsing in description files

          Patrick Connolly added a comment - - edited

          OK. Seems the fix that would all escaping of Token Macros happened in Feb 2012, but no release yet
          https://github.com/jenkinsci/token-macro-plugin/compare/da2858d...38919bf

          So it seems that any "$" in descriptions will fail the build, so a toggle in project-description-setter would be AMAZING

          Thanks!

          Patrick Connolly added a comment - - edited OK. Seems the fix that would all escaping of Token Macros happened in Feb 2012, but no release yet https://github.com/jenkinsci/token-macro-plugin/compare/da2858d...38919bf So it seems that any "$" in descriptions will fail the build, so a toggle in project-description-setter would be AMAZING Thanks!

          Code changed in jenkins
          User: bap2000
          Path:
          src/main/java/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapper.java
          src/main/java/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapperDescriptor.java
          src/main/resources/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapper/config.jelly
          src/main/resources/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapper/config.properties
          src/main/resources/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapper/global.jelly
          src/main/resources/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapper/global.properties
          src/main/resources/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapper/help-disableTokens.html
          src/main/resources/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapper/help-projectDescriptionFilename.jelly
          http://jenkins-ci.org/commit/project-description-setter-plugin/d021f9e62755f88ffd7d0de122ee65093f0c5db9
          Log:
          [FIXED JENKINS-14088] add option to disable token parsing

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: bap2000 Path: src/main/java/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapper.java src/main/java/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapperDescriptor.java src/main/resources/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapper/config.jelly src/main/resources/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapper/config.properties src/main/resources/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapper/global.jelly src/main/resources/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapper/global.properties src/main/resources/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapper/help-disableTokens.html src/main/resources/org/jenkinsCi/plugins/projectDescriptionSetter/DescriptionSetterWrapper/help-projectDescriptionFilename.jelly http://jenkins-ci.org/commit/project-description-setter-plugin/d021f9e62755f88ffd7d0de122ee65093f0c5db9 Log: [FIXED JENKINS-14088] add option to disable token parsing

          bap added a comment -

          New (advanced) option to disable token parsing in the job config.
          Option in global config to set the default value for new jobs.

          bap added a comment - New (advanced) option to disable token parsing in the job config. Option in global config to set the default value for new jobs.

            bap bap
            patcon Patrick Connolly
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: