sebastianbergmann/diff

Diff wrapper with some utils

soullivaneuh opened this issue · 1 comments

soullivaneuh commented

Instead of returning a simple array of Diff for Parser::parse, we may return a DiffWrapper object with some functionalities:

  • Make it iterable to work as an array. Plus, this may keep BC.
  • Add a getPosition method which return the diff position of a given path and line number, if exist.
  • Maybe more? Just tell! :-)

WDYT? I can propose a POC PR.

sebastianbergmann commented

I would like to keep this component as simple as possible.