refactor(lines_filter_map_ok): move to under methods/

[ada4a]

changelog: none

[rustbot]

r? @Jarcho

rustbot has assigned @Jarcho.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

[Jarcho]

Can you make the first commit strictly a file rename. It helps significantly with merge conflicts.

[ada4a]

Done. I was hesitant to have a broken intermittent commit, but the diff does come out a bit nicer

[Jarcho]

Thank you.

View changes since this review

[Jarcho]

Done. I was hesitant to have a broken intermittent commit, but the diff does come out a bit nicer

It's generally preferred if they pass tests, but that's not a hard rule and nothing actually breaks (bisect needs to use --first-parent). File renaming is one of the biggest reasons to ignore that rule.