Closed Bug 921450 Opened 11 years ago Closed 11 years ago

Enable IRC notifications from Travis

Categories

(L20n :: JS Library, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: stas, Unassigned)

Details

Attachments

(1 file)

In bug 881704 I enabled Travis CI, but I couldn't turn IRC notifications on back then due to a bug on the Travis side: https://github.com/travis-ci/travis-ci/issues/436

The bug has been recently fixed, so we can now turn notifications on.  They look like this:

14:10 -!- travis-ci [travis-ci@moz-221B84B3.compute-1.amazonaws.com] has joined #l20n
14:10 -travis-ci:#l20n- [travis-ci] stasm/l20n.js#97 (irc-notify - 791d02b : Stas Malolepszy): The build passed.
14:10 -travis-ci:#l20n- [travis-ci] Change view : https://github.com/stasm/l20n.js/commit/791d02b12c6c
14:10 -travis-ci:#l20n- [travis-ci] Build details : http://travis-ci.org/stasm/l20n.js/builds/11862651
14:10 -!- travis-ci [travis-ci@moz-221B84B3.compute-1.amazonaws.com] has left #l20n []

And they can be customized as described here: http://about.travis-ci.org/docs/user/notifications/#IRC-notification
Attached patch PatchSplinter Review
Attachment #811091 - Flags: review?(gandalf)
Attachment #811091 - Flags: review?(gandalf) → review+
https://github.com/l20n/l20n.js/commit/65ba85e23d4736ed572b7d5f3cf3e6d36e0cd42e
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: