Dinocatalog

Rubocop said Yes
There's a lot wrong with this method. First, the name. It suggests a boolean (e.g. is the dinosaur a carnivore), but is named `carnivore` & not `carnivore?`. Which is easy to ignore since it doesn't return a boolean, but rather a string. My recommendation there is to name the method after the attribute of your Dinosaur class, `diet`.

Next, generically named variables such as `value` are a pretty distinct code smell. There are times it makes sense, for instance if you're doing some generic iteration over a list, then it might be ok. This is clearly not that, & it should probably be renamed something like `is_a_carnivore`.