search

dnmfarrell / Perly-Bot

a RSS trawler and social media broadcaster
BSD 2-Clause "Simplified" License
15 stars 6 forks source link

Keywords/strings too broad? #21

Open perlancar opened 8 years ago

perlancar commented 8 years ago

I personally think the set of keywords and strings are too broad:

https://github.com/dnmfarrell/Perly-Bot/blob/master/lib/Perly/Bot.pm#L202

my $looks_perly =
    qr/\b(?:perl|perl6|cpan|cpanm|moose|metacpan|module|timtowdi|yapc|\:\:)\b/i;

Specifically, module and :: are not very Perl-specific.

briandfoy commented 8 years ago

We tend to pull from Perl blogs, so there is already a context. The problem is that this is applied to the title and description instead of the actual content. I'm moving things around so we can fetch the actual posts and look that those, then apply stricter logic.