-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrate to caniuse-lite. #57
Comments
I think |
I did some additional research into the For example, https://github.com/ZoomZhao/canidiff uses both browserslist and caniuse-db; with browserslist moving its source of truth to caniuse-lite, this module will have to load the data essentially twice; ping @ZoomZhao. So based on this my conclusion is that we should add the |
What about letting people choose between one or the other? Anyway, node_modules for real world projet is never less than 100MB so not sure it's super important. |
@MoOx nobody block you from using Many users complains about |
The problem with allowing choice of dataset is that people will want to choose different options and users will be stuck with even more to download; both That's why I would like us to unify caniuse tools around this smaller dataset, which is specifically designed for module authors and use cases that need only the statistics that I am currently working on a fully automated publishing/testing solution for |
@ben-eb Could caniuse-db consume caniuse-lite as a dependency? Maybe that would keep both parties happy. |
@bebraw unfortunately, So there is no way to change something in |
@ai Ok, sad to hear. Big 👍 for |
|
@Nyalab your comments will be very helpful here. |
Hello everyone, sorry for the delay, i was moving houses, it is over now. It is ok for me to switch on |
@Nyalab Cool, planning to start automated releases tomorrow, will send a PR just before this. 👍 |
@agauniyal Yep, I think I will recommend a major release of caniuse-api because of the data source migration. However the surface API does not need to be changed. 😃 I'm waiting on a caniuse-db release to see if my publish script is working as intended and then I can recommend switching. |
Hi @Nyalab!
I've been working on an unofficial way to consume caniuse support data that is intended for modules, whereas the official dataset is intended for people. So I dropped a lot of keys that Autoprefixer & Browserslist don't use and then compacted the rest of the data. The end result weighs around 7 times less than the official database as of this writing, and the only downside is a slight performance hit when the data is restored into a caniuse format. We optimised for size, not speed, but I think we can perhaps improve this further in the future. For now the dataset is as close to caniuse-db as possible, for easy migration.
So I think it's important to use a single source of truth for the support data, and as I am starting to make much more use of caniuse-api I'd like to update the module to use caniuse-lite too.
My only reservation about the migration is that you are using the
title
key from each feature. I'd like to know if that can be dropped so that we can keep caniuse-lite as lean as possible. But I would like some input from caniuse-api users, on whether they think this is necessary or not. I'm open to any feedback! 😄Copying in users of this module/caniuse data: @agauniyal, @una, @schiehll, @sylvainpolletvillard, @MoOx, @bebraw & @ai.
Thanks!
The text was updated successfully, but these errors were encountered: