Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: c7a5bcbd7bb81bdb04d0cdaf9c4a29d67c46c4e1
      
https://github.com/Perl/perl5/commit/c7a5bcbd7bb81bdb04d0cdaf9c4a29d67c46c4e1
  Author: Lukas Mai <lukasmai....@gmail.com>
  Date:   2023-08-08 (Tue, 08 Aug 2023)

  Changed paths:
    M .github/workflows/irc-notifications.yaml

  Log Message:
  -----------
  disable duplicate IRC bot notifications on #p5p

Currently every new pull request generates seven lines of activity in
irc.perl.org/#p5p that look like this:

    <botifico> [perl5] khwilliamson opened pull request #21302: autodoc: A 
config.h entry may not clash with perlintern - 
https://github.com/Perl/perl5/pull/21302
    -!- Pull-Request [~pull-requ@40.84.179.161] has joined #p5p
    <Pull-Request> khwilliamson opened PR #21302
    <Pull-Request> autodoc: A config.h entry may not clash with perlintern
    <Pull-Request> https://github.com/Perl/perl5/pull/21302
    <Pull-Request>
    -!- Pull-Request [~pull-requ@40.84.179.161] has left #p5p []

Six of those are redundant because botifico already handles everything.

This commit gets rid of the empty message by stripping trailing newlines
from the YAML string. It also tries to fix the "merged PR" and "closed
PR" cases that currently don't work at all.

It also disables #p5p IRC notifications by default. They can be enabled
by setting the github project variable NOTIFY_IRC_P5P to 'true'.


Reply via email to