PR#56: Add notification for badges to #fedora-badges - fedora-infra/ansible

fedora-infra / ansible

#56 Add notification for badges to #fedora-badges

Merged 3 years ago by pingou. Opened 3 years ago by pingou.

fedora-infra/ pingou/ansible badges into master

Add notification for badges to #fedora-badges

Pierre-Yves Chibon • 3 years ago

4f04aa7

roles/fedmsg/irc/templates/ircbot.py

file modified

+22

		`@@ -614,6 +614,28 @@`
		`],`
		`),`
		`),`
		`+`
		`+ # Hook up #fedora-badges with badges messages`
		`+ dict(`
		`+ network='chat.freenode.net',`
		`+ port=6667,`
		`+ make_pretty=True,`
		`+ make_terse=True,`
		`+`
		`+ {% if env == 'staging' %}`
		`+ nickname='fm-stg-badges',`
		`+ {% else %}`
		`+ nickname='fm-badges',`
		`+ {% endif %}`
		`+ channel='#fedora-badges',`
		`+ filters=dict(`
		`+ topic=[`
		`+ '^((?!(pagure.(new\|added)\|mailman)).)$',`
		`+ ],`
		`+ body=['^((?!(fedora-badges\|badges)).)*$'],`
		`+ ),`
		`+ ),`
		`+`
		`],`

		`### Possible colors are ###`

pingou commented 3 years ago

Signed-off-by: Pierre-Yves Chibon pingou@pingoured.fr

rebased onto 4f04aa7

3 years ago

Metadata Update from @pingou:
- Request assigned

3 years ago

jflory7 commented 3 years ago

Huzzah. :tada: Looks good to me! I have merge rights here, but should I go ahead and merge, or is there a process to follow here?

pingou commented 3 years ago

I can't promise it does everything you want but it's a start, we can try tweaking afterward.

jflory7 commented 3 years ago

Yep. We can share any feedback after putting it in use for a week or two. Anyways, I give a +1 on merging but I don't want to press the big green button unless someone else okay's it first. :grinning:

pingou commented 3 years ago

Let's get it in and out :)

Pull-Request has been merged by pingou

3 years ago