why don't we call it just Flags so it will be builder.Flags? and also the functions below like NewFlags instead of NewBFlags etc etc (and filename to just builder/flag.go :))

Since we have bflag.go, I thought we already consider bflag for short of buildflag, and bflag would be more meaningful than just flag. Personally I don't like that common word as a variable or method's name, it's painful when we jump between these tags 😞

If you insist, I surely can change the name 😄

Fair enough 😄

How about FlagSet?

I tend to agree with @hqhq - having 'b' or something (like "builder") in there to make it clear its not the same "flags" as the cmd line stuff is useful. But no biggie either way.

@duglin That's the point of the doc string :)

s/containers/contains/

s/define constant/Define constants/

Pretty sure this should be private. Let's fix this. Then you don't need docstring at all :)

Right, didn't check it.