On 26/03/11 05:48, Daniel Friesen wrote:
What about the fixmes left open since it's not clear if anything is even still broken currently.
If it is unclear: it either need a clarification or deserve a reversion. We already have enough lines hiding in the fog, read to jump at you when you get out of the path.
The fixmes for things like extra things like new tests should be added, but the actual commit in question isn't broken in any way.
The fixmes for things which are perfectly functional, but need a minor bit of tweaking since they work perfectly find, but don't use the best practice methods to do it.
Do we even have fixmes for the last two cases? Anyway for tests, they might be required just to make sure other developers using the feature will use it as intended. There are always funny corner cases to handle, specially with PHP.