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

Developer can see ANSI color in log file

    Details

    • Similar Issues:
    • Sprint:
      Blue Ocean 1.2-beta3, Blue Ocean 1.2-beta4, Blue Ocean 1.2

      Description

      Improvement on roadmap

      This improvement is on the Blue Ocean project roadmap. Check the roadmap page for updates.

      Has been requested that support for ANSI colours is desired

      Notes

        Attachments

          Issue Links

            Activity

            Hide
            michaelneale Michael Neale added a comment -

            Andrew Gray remove the ansiColor, and colour codes should automatically show up I believe right Josh McDonald

             

            ansiColor injects binary noise and wipes out the real ansi codes so can't work with blue ocean or any log viewed other than the jenkins log, due to a very bad design mistake early on. 

            Show
            michaelneale Michael Neale added a comment - Andrew Gray  remove the ansiColor, and colour codes should automatically show up I believe right Josh McDonald ?    ansiColor injects binary noise and wipes out the real ansi codes so can't work with blue ocean or any log viewed other than the jenkins log, due to a very bad design mistake early on. 
            Hide
            sophistifunk Josh McDonald added a comment -

            Yes, currently you need to run your pipeline without ansiColor in order to see the colours in BO.

            Show
            sophistifunk Josh McDonald added a comment - Yes, currently you need to run your pipeline without ansiColor in order to see the colours in BO.
            Show
            jglick Jesse Glick added a comment - https://github.com/jenkinsci/ansicolor-plugin/pull/132#issuecomment-440937753
            Hide
            sophistifunk Josh McDonald added a comment - - edited

            Happy to see this! It's way more thorough than my experimental patch I never had the time to fix tests for  All I did was stop stripping the codes, and bookend them with injected

            <!--

            and

            -->

            "tags" in the annotation stream. 

            Show
            sophistifunk Josh McDonald added a comment - - edited Happy to see this! It's way more thorough than my experimental patch I never had the time to fix tests for   All I did was stop stripping the codes, and bookend them with injected <!-- and --> "tags" in the annotation stream. 
            Hide
            jglick Jesse Glick added a comment -

            Ooh, the use of comments rather than <span style="display: none">…</span> is a nice trick. Ported it

            Show
            jglick Jesse Glick added a comment - Ooh, the use of comments rather than <span style="display: none">…</span> is a nice trick. Ported it

              People

              • Assignee:
                sophistifunk Josh McDonald
                Reporter:
                michaelneale Michael Neale
              • Votes:
                29 Vote for this issue
                Watchers:
                43 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: