National Post -> Postmedia

[geofferb]

First fixed the NP translator, so the tests pass again.

Since all newspaper sites owned by Postmedia (parent of National Post) use the exact same layout, I've expanded the translator to include the 103 other newspapers and websites that Postmedia owns based on the list at https://www.postmedia.com/brands/. I've included some tests to confirm it works on these other websites.

[AbeJellinek]

Thanks! Nice regex :)

Tests and lint should work if you rebase on master.

I'm hesitant to merge translators that are basically just JSON-LD parsers, because I really would like to make them unnecessary by finishing up #1897 soon. But this adds support for so many sites at once that I do think it makes sense to merge now.

[geofferb]

@AbeJellinek

Thanks! Nice regex :)

I probably put a bit too much time into writing that :p

Tests and lint should work if you rebase on master.

Rebased on master and did a force push, and it's still failing; am I missing something?

I'm hesitant to merge translators that are basically just JSON-LD parsers, because I really would like to make them unnecessary by finishing up #1897 soon. But this adds support for so many sites at once that I do think it makes sense to merge now.

I wondered if there was a universal JSON-LD translator while working on this as well, glad to hear one is being worked on!