Skip to content

Conversation

@jglick
Copy link
Member

@jglick jglick commented Oct 29, 2018

Copy link
Member

@dwnusbaum dwnusbaum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix looks good to me. Would be nice to add a test if it is easy to do so.

@jglick
Copy link
Member Author

jglick commented Oct 30, 2018

I looked and there is no existing test coverage mentioning the word console, so that would be new development. Would indeed be desirable. TBD if I will have time to write it.

@cyrille-leclerc
Copy link
Contributor

Thanks @jglick and @dwnusbaum

@cyrille-leclerc cyrille-leclerc merged commit 3125357 into jenkinsci:master Oct 31, 2018
@jglick jglick deleted the remote-console-notes-JENKINS-54133 branch November 1, 2018 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants