[JENKINS-9687] log build step which has changed the build result to console

[kutzi]

Logs whenever a build step changes the build result to the console.

Note: I've also added a getName() method the the BuildStep interface to be able to log a more descriptive name than just the BuildStep's class name. If that change is to heavy weight in your opinion we can also just use e.g. the simple class name.

[orrc]

Yes! Great idea.

Can we not just use Descriptor.getDisplayName() by default, since that's the human-readable name that users see in the Configure page? i.e. "The 'Archive the artifacts' build step changed the result" seems more useful than "Build step 'ArtifactArchiver' changed the result"

Also, many of the "inheritDoc" Javadoc tags seem to have two closing braces. Probably this could just be one line of Javadoc (plus the annotation):
/** {@inheritdoc} */
@OverRide

[kutzi]

Good idea with the descriptor. I think, I'll just check if it's instanceof Describable. if yes, return getDisplayName(), otherwise getClass().getSimpleName(). That would make this a much 'lighter' change.

[kutzi]

I wanted to create a new pull request, but seems Github is intelligent enough to automatically take the new commit :)

[kutzi]

Okay, since this doesn't include any interface changes anymore, I felt confident enough to push it on myself ;)