Why does `DiskCache` need to impl `Display`?

[Diptesh Choudhuri]

Looking at the docs for DiskCache here, it defines:

impl<K, V> DiskCache<K, V>
where
    K: Display,
    V: Serialize + DeserializeOwned

Is there any particular reason why it needs to implement Display?

[Diptesh Choudhuri]

If possible, could this be removed completely? If not, atleast be gated behind a feature?

[Diptesh Choudhuri]

@jaemk Could you take a look at this please?

[Omid Rad]

@IgnisDa I think it'll move it faster, if you create a PR.

[Diptesh Choudhuri]

I tried to do it here but it did not work: https://github.com/IgnisDa/cached/tree/remove-display-trait

[Omid Rad]

@IgnisDa I think just replacing Display with ToString will do the job.

[Diptesh Choudhuri]

@omid Made a PR.