search

TkTech / notifico

IRC Message Relay
http://n.tkte.ch
MIT License
155 stars 39 forks source link

Ignore travis pending messages #94

Closed Fohlen closed 5 years ago

Fohlen commented 10 years ago

There should be a option to ignore pending messages sent by travis because they're annoying spam.

Dav1dde commented 10 years ago

Travis plugin or Github status Hook?

Fohlen commented 10 years ago

Travis plugin. In IRC they're looking like this:

Not-ad92: [16:56:08] [sauerbraten-fork] Pending. The Travis CI build is in progress - https://travis-ci.org/sauerbraten-fork/sauerbraten-fork/builds/35284200

Nowaker commented 9 years ago

I use Travis CI hook. This is all I get:

23:45:40 < Not-7e7f> [virtkick-webapp] Travis CI - build #201 broken. (master @ 51f50a3) http://git.io/YzP2Rw
23:45:41 < Not-7e7f> [virtkick-webapp] Details: https://travis-ci.org/virtkick/virtkick-webapp/builds/44627019

I don't get any "Pending" messages so I believe this can be closed.

TkTech commented 9 years ago

Can anyone else confirm @Nowaker's observation? I don't use Travis, so I don't know if this was already resolved.

Mikaela commented 9 years ago

I am using GitHub status and it gives me "pending" message twice and the first time appears to be to tell that the build has been added to the queue and the second that the build has been started.

Croydon commented 9 years ago

It's really annoying to get sometimes more pending messages than anything else. Please do something about more filter possibilities in general.

dequis commented 9 years ago

"Pending. The Travis CI build is in progress" really looks like the status part of the github hook.

So, just configure the events in github's hook to not send "status", then switch to the travis-ci hook?

Mikaela commented 9 years ago

So, just configure the events in github's hook to not send "status", then switch to the travis-ci hook?

Not very good idea. https://github.com/travis-ci/travis-ci/issues/1094

dequis commented 9 years ago

Hm, I see. How about adapting PR #116?

qreeves commented 9 years ago

Confirmed this bug still exists: 19:56:42 <Not-f107> [base] Pending. The Travis CI build is in progress - https://travis-ci.org/red-eclipse/base/builds/49930941 19:57:12 <Not-f107> [base] Pending. The Travis CI build is in progress - https://travis-ci.org/red-eclipse/base/builds/49930941

jlu5 commented 9 years ago

Also confirming; for some reason I get around 5 pending messages for my tests (6 Python versions configured, 2 in allow_failures), and 2 success messages...

qreeves commented 9 years ago

I've since determined that the Travis hook isn't actually necessary. If setup correctly, these messages can be sent from the normal GitHub hook by selecting the correct events to trigger on (as Travis integrates with GitHub already to notify it of build success/failure). I'm now using Semaphore with this setup and everything works perfectly.

Fohlen commented 5 years ago

Appears to be related to GitHub and not notifico. Closing.