Page MenuHomeSoftware Heritage

Use pkg_resources to get the package version instead of vcversioner
ClosedPublic

Authored by olasd on Nov 21 2019, 6:26 PM.

Details

Summary

Planned changes because I need to figure out where to put a user-agent
check before landing this.

T2105.

Test Plan

tox

Diff Detail

Repository
rDLS Listers
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont added a subscriber: ardumont.

It's currently untested and it's not really clear how to test this without duplicating the "potential" test in each lister.
Given that, i would not be shocked if for once, you push that as is.

Having said that, it comes to mind that there might be one lister which seems simpler than the other and use directly requests: cgit lister.
Although it does not use that __version__, you could make it so and add a test for it.

What do you think?

And thanks for fixing it.

Cheers,

This revision is now accepted and ready to land.Nov 22 2019, 12:04 PM
olasd retitled this revision from [WIP] Use pkg_resources to get the package version instead of vcversioner to Use pkg_resources to get the package version instead of vcversioner.
This revision is now accepted and ready to land.Nov 22 2019, 12:08 PM