Last modified: 2014-10-15 12:49:46 UTC
ExternalChangesList has no documentation on what it's supposed to be and class-orientated (ie everything is static). It also has a 120+ line method that should probably be split up.
It also has no tests.
From the looks of it, it's a utility class for constructing changes entries on on RecentChanges and the watchlist. Unit testing this will probably just be a bunch of regular expressions against the output, but we should have that anyway.
Still relevant?
Yes. Looks like this code has not been improved upon. Also still lacks tests for basically everything.