search

web-platform-tests / wpt.live

A live version of the web-platform-tests project
https://wpt.live/
15 stars 11 forks source link

Convert from requirements.txt to pipenv's Pipfile #50

Closed foolip closed 3 years ago

foolip commented 3 years ago

This makes it much clearer what our own dependencies are and which are transitive dependencies.

Upgrade flake8 to make it actually work on Python 3.8 locally, and lint all of the Python code, fixing the errors that remained.

Closes https://github.com/web-platform-tests/wpt.live/pull/46.

foolip commented 3 years ago

@past r? It quickly became clear in https://github.com/web-platform-tests/wpt.live/pull/44 + https://github.com/web-platform-tests/wpt.live/pull/46 + https://github.com/web-platform-tests/wpt.live/pull/47 that tracking transitive dependencies directly results in having to upgrade them together, and this makes that possible. (Using requirements.in + requirements.txt would also allow for it, but I wanted to try Pipenv in a low-risk repo, and this is just linting.)

past commented 3 years ago

Much simpler, thanks!