Use sparse index

[BarbossHack]

fixes #197

[serayuzgur]

Please create an new file under api like sparse-index-server.ts and implement logic under it. We should bind it to a configuration which one to use. For example if server adress is empty extension can use sparse implementation.

[simonsan]

I agree, I think it would be good to keep both implementations and switch depending on the users' setting. Like that, serayuzgur can still implement their use case, while most other people would use the sparse index.

[BarbossHack]

Please create an new file under api like sparse-index-server.ts and implement logic under it. We should bind it to a configuration which one to use. For example if server adress is empty extension can use sparse implementation.

I've pushed new changes, following your advice :

• new implementation in sparse-index-server.ts
• the sparse index is the default if indexServerURL is empty, else it will use what user put in indexServerURL (using crates-index-server.ts)
• both implementation are now using CrateMetadatas strong type, to ensure they all return the same data structure
• StatusBar now display the current indexServerURL domain name

[serayuzgur]

StatusBar now display the current indexServerURL domain name @BarbossHack Isn't it too verbose

[BarbossHack]

Isn't it too verbose

Before it was Fetching crates.io, now it's Fetching index.crates.io by default, and else (for example) Fetching api.crates-vsc.space, it should be ok imho. But we could also change it to Fetching index

[simonsan]

StatusBar now display the current indexServerURL domain name @BarbossHack Isn't it too verbose

As a user, I like the addition, and don't find it verbose. It should be limited to the third-level.second-level.top-level domains, though.

[serayuzgur]

@BarbossHack thanks , i will test it and publish .

[BarbossHack]

I've pushed changes, to fix https://github.com/serayuzgur/crates/issues/202 for sparse-index fetching (see toLowerCase in sparse-index-server.ts)

[CAD97]

Is there an expected timeline for releasing this?

[serayuzgur]

@CAD97 Just published 0.6.2