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'.