Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JENKINS-37284] Added custom pipeline step #83

Merged
merged 3 commits into from
Nov 7, 2016

Conversation

cpoenisch
Copy link
Contributor

@cpoenisch cpoenisch commented Nov 4, 2016

Relates to issue #80 and JENKINS-37284.

Provides the ability to use a custom pipeline step without allocating a node and a separate build wrapper.

Instead of

node {
  wrap([$class: 'AnsiColorBuildWrapper', 'colorMapName': 'XTerm']) {
    <something that outputs ansi colored stuff>
  }
}

also following pipeline syntax can be used

ansiColor('xterm') {
  <something that outputs ansi colored stuff>
}

@dblock
Copy link
Member

dblock commented Nov 4, 2016

Looks good to me, @ejelly @qvicksilver also take a look pls?
Needs CHANGELOG and README updates, please.

@dblock
Copy link
Member

dblock commented Nov 7, 2016

Looks great, merging.

@cpoenisch Want to help out with the next release?

@lsglick
Copy link

lsglick commented Nov 14, 2016

Hi @dblock! Any idea when we'll see this in a new release? Thanks!

@dblock
Copy link
Member

dblock commented Nov 15, 2016

I was hoping someone else can do it, see #75. Add your +1.

@eyelock
Copy link

eyelock commented Nov 20, 2016

Also added my +1 to this, having the coloring in a non-node context would be pretty cool. Would love to do a release but pretty new to all this (teach a man to fish though ...)

@dblock
Copy link
Member

dblock commented Nov 21, 2016

I tried to make a release, but right now maven.jenkins-ci.org 8081 seems MIA :(

Will try later ...

@dblock
Copy link
Member

dblock commented Nov 21, 2016

OK, that host was discontinued, see https://twitter.com/jenkinsci/status/800526772773142530
I updated the pom.xml with the correct URL in 38ae5ec

@dblock
Copy link
Member

dblock commented Nov 21, 2016

This was finally released in 0.4.3. Thanks for your patience.

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.

4 participants