laminas-feed still relevant for this component?

[froschdesign]

Bug Report

Q	A
Version(s)	3 & 4

Summary

laminas-feed is listed as required package for developing but is this really need?

Branch 4.0.x:

laminas-cache/composer.json

Line 52 in 2db8ef0

"laminas/laminas-feed": "^2.20",

Branch 3.13.x:

laminas-cache/composer.json

Line 56 in 66cf9a2

"laminas/laminas-feed": "^2.20",

[boesing]

I'd say no. I guess that was the case due to autocompletion in documentation. if laminas-feed is not mentioned in the docs anymore, we can remove that dependency.
if it is, we might want to think about moving the documentation to the feed component instead.

[Xerkus]

There is no mention of feed in the package whatsoever:

[laminas-cache (3.11.x)]$ ag feed
composer.json
56:        "laminas/laminas-feed": "^2.20",

composer.lock
<snip>

Adapters also require laminas-feed but they have no occurrences of "feed" in either source or tests.

[froschdesign]

if laminas-feed is not mentioned in the docs anymore, we can remove that dependency.

Correct, there was an example in the documentation of version 2:

https://github.com/laminas/laminas-cache/blob/2.13.x/docs/book/pattern/class-cache.md#examples

if it is, we might want to think about moving the documentation to the feed component instead.

The example is for the removed class cache feature, so it is no longer needed.
(But the description is missed in version 2 of the laminas-cache documentation. 🤔)

@Xerkus @boesing
Thanks for your answers!

[froschdesign]

Fixed with #291