search

mattbaird / elastigo

A Go (golang) based Elasticsearch client library.
Apache License 2.0
943 stars 243 forks source link

Cluttered signatures #18

Open lino opened 11 years ago

lino commented 11 years ago

Hey, not really an issue, but is there any reason for passing the pretty bool? I find this confusing, as the json is parsed anyway. I'll refactor it out (into a single setting if needed), but it will break things for some users.

mattbaird commented 11 years ago

Hi linohh - Yes please refactor it out to a setting. I originally put it in there to make debugging a bit easier and to be honest got a bit lazy when I should have removed it. Thanks.