▲ | khasan222 16 hours ago | |||||||
That codebase definitely leaves much to be desired, I’ve already had to fork it for work in order to fix some bugs. 1 such bug, find a foreign language with commas in between numbers instead of periods, like Dutch(I think), and a lot of prices on the page. It’ll think all the numbers are relevant text. And of course I tried to open a pr and get it merged, but they require tests, and of course the tests don’t work on the page Im testing. It’s just very snafu imho | ||||||||
▲ | fabrice_d 5 hours ago | parent [-] | |||||||
This seems to be https://github.com/mozilla/readability/pull/853#issuecomment... and I think their expectations are pretty reasonable. | ||||||||
|