Hey @akx I have address the issues you mentioned in #792 . There are now tests, the style is more in line with your code and line numbers are handled correctly. Would it be possible to merge this or do you need me to do something more?
PS: I will try to contact you in some other way as well, given that you wrote somewhere here that you muted github notifications.

Codecov Report

Merging #939 (8a6c452) into master (82c41cc) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #939      +/-   ##
==========================================
+ Coverage   91.56%   91.62%   +0.05%     
==========================================
  Files          23       23              
  Lines        4244     4273      +29     
==========================================
+ Hits         3886     3915      +29     
  Misses        358      358              

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Hey @akx ,
is there anything you need me to do or does this look ok for merging?

@akx OK, all changes made as you requested.

As for:

Also, maybe it should be in options all in all, so as not to change the extractor function signature?

This is up to you to decide. I can see arguments doing either (it is an option, but it's different from the other ones). Feel free to make that change if you think it's better to do it that way. I have left it as it was for now.

Please let me know if you need me to do anything else so that we can get this merged ASAP!