--loglevel should be --log-level

alephcloud / hs-yet-another-logger

Haskell logging framework with a focus on flexibility and performance

Other

5 stars 4 forks source link

--loglevel should be --log-level #14

Closed jonsterling closed 9 years ago

jonsterling commented 9 years ago

I can submit a pull request to this effect soon.

larskuhtz commented 9 years ago

Cool.

I wonder if we should keep the current version as deprecated legacy syntax around (possible hidden in the output of --help)? Not sure if it's worth though.

jonsterling commented 9 years ago

I guess it would depend on whether anybody is using this package yet for anything where someone depends on the old syntax...

larskuhtz commented 9 years ago

I use yet-another-logger in a few places in other packages, but I don't think that I have code depending on this syntax. Also I don't think anyone else depends on it. Yet, probably we should increase the second level version number if we remove the old syntax. I think it's fine.

larskuhtz commented 9 years ago

Solved in PR #21