browserslist / update-db

CLI tool to update caniuse-lite to refresh target browsers from Browserslist config
https://browsersl.ist/
MIT License
286 stars 57 forks source link

Speed up `update-db` tests #23

Open ialarmedalien opened 3 years ago

ialarmedalien commented 3 years ago

The update-db tests use repos with quite a few dependencies (autoprefixer, postcss-*...). The tests could run more quickly if some less dependency-heavy repos were used -- e.g. caniuse-on-user-agent, caniuse-support, caniuse-api, browserslist-ga. Alternatively, if you want to keep everything "in house", create some npm packages with a bare minimum of content and dependencies (including caniuse-lite) and use those.

ai commented 3 years ago

I agree. It is a good idea. You can add it to your PR (Sorry, I will not be able to review your PR today).

ialarmedalien commented 3 years ago

No problem on not reviewing the PR today. Do you have a preference for which strategy is used -- existing repos vs create testing use only npm modules?

ai commented 3 years ago

I think it will be better to use real dependencies