search

JEG2 / highline

A higher level command-line oriented interface.
Other
1.29k stars 137 forks source link

Constants being redefined on lib/highline/builtin_styles.rb #166

Closed abinoam closed 9 years ago

abinoam commented 9 years ago

Hi @matugm ,

rake acceptance shows these warnings.

Could you have a look at it? I think it's related to the recent PRs.

/Users/abinoam/src/highline/lib/highline/builtin_styles.rb:30: warning: already initialized constant HighLine::BuiltinStyles::STYLES
/Users/abinoam/src/highline/lib/highline/builtin_styles.rb:9: warning: previous definition of STYLES was here
/Users/abinoam/src/highline/lib/highline/builtin_styles.rb:71: warning: already initialized constant HighLine::BuiltinStyles::COLORS
/Users/abinoam/src/highline/lib/highline/builtin_styles.rb:32: warning: previous definition of COLORS was here
matugm commented 9 years ago

Yeah, the problem is that I had to define the constant twice, one for legacy use and one for the new format of defining the STYLES. I will rename the new one to something else and that should get rid of those warnings.