cowsay icon indicating copy to clipboard operation
cowsay copied to clipboard

Wrong example in README

Open jcubic opened this issue 5 years ago • 3 comments

One of the examples shows:

cowsay.say({
	text : "I'm a moooodule",
	e : "oO",
	T : "U "
})

which does't work (eyes and tongue don't change), other example says:

console.log(think({
  text: 'grazing in the browser',
  cow: SQUIRREL,
  eyes: 'pp',
  tongue: ';;',
}));

that API works.

jcubic avatar Jun 25 '20 11:06 jcubic

in my case

cowsay.say({
	text : "I'm a moooodule",
	e : "oO",
	T : "U "
})

works

But

console.log(think({
  text: 'grazing in the browser',
  cow: SQUIRREL,
  eyes: 'pp',
  tongue: ';;',
}));

no. And the cow: change animal option not working in neither cases // still confused how to change animal successfully

Questionm avatar Jun 30 '20 01:06 Questionm

There are some inconsistencies, I think the best is to provides both APIs for both say and think. Here is was testing the APIin broser at CodePen

jcubic avatar Jun 30 '20 07:06 jcubic

I've just tested both say and think at CodePen and in browser it works with longer names options. They eyes are changed.

jcubic avatar Jun 30 '20 07:06 jcubic