Resolve relative paths when adding sources

plasma-umass / slipcover

Near Zero-Overhead Python Code Coverage

Apache License 2.0

475 stars 17 forks source link

Resolve relative paths when adding sources #42

Closed peterjc closed 5 months ago

peterjc commented 1 year ago

Enables use of .. in --source arguments.

Closes #41

jaltmayerpizzorno commented 1 year ago

Thank you! Would you mind creating a test for it?

peterjc commented 1 year ago

Would you lean to a more complex tree under tests/, or an example path structure built on the fly under /tmp?

jaltmayerpizzorno commented 1 year ago

Mm, good question. Probably tmp would be best. tests is getting a bit crowded. Thank you!

peterjc commented 1 year ago

It might be quicker for you to take over the PR and write a test, this was a quick drive-by exploration of the tool late last night, and I have too many more pressing tasks this week to give this one justice. Sorry.

jaltmayerpizzorno commented 1 year ago

No problem, thank you for contributing the change (and bug report)! I'll leave open until I can work on it.