Ticket #852 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

irc status failure or failed?

Reported by: ccomb Owned by:
Priority: minor Milestone: undecided
Version: 0.7.12 Keywords:
Cc:

Description

 http://djmitche.github.com/buildbot/docs/0.7.12/#IRC-Bot

The doc says : failed

A build failed

You should replace 'failed' with 'failure' in the doc.

As a consequence of this documentation error, I've first tried putting 'failed' in the notify_events configuration parameter of the IRC status, and I did't get a configuration error, but an error at runtime, which is difficult to understand. I've had to modify the source code and replace the UsageError? with a log.msg in validate_notification_event to be able to understand the configuration error.

It would be nice to change the way this configuration error is handled.

Change History

comment:1 Changed 3 years ago by Dustin J. Mitchell

  • Status changed from new to closed
  • Resolution set to fixed

fix IRC docs: change failed -> failure (fixes #852)

I don't see a quick way to fix the usage error; this should clean things up a bit.

Changeset: d413c69ab3a3108b9a78f5694cb4dde87c510ba4

comment:2 Changed 3 years ago by Dustin J. Mitchell

fix IRC docs: change failed -> failure (fixes #852)

I don't see a quick way to fix the usage error; this should clean things up a bit.

Changeset: d413c69ab3a3108b9a78f5694cb4dde87c510ba4

Note: See TracTickets for help on using tickets.