search

cncf-tags / container-device-interface

Apache License 2.0
216 stars 39 forks source link

pkg/cdi: drop unused return value from NewCache function #188

Closed marquiz closed 10 months ago

marquiz commented 10 months ago

Drop the unused (always nil) error returned by NewCache(). This eliminate some unreachable error paths in the code.

NOTE: This change changes the exported API of "pkg/cdi" package as the prototype of NewCache() changes.

marquiz commented 10 months ago

187 was merged

marquiz commented 10 months ago

Thanks for the reviews. I got stuck pondering that if we change the cache API should we change registry API correspondingly, or not?