Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

API change: remove field CommandLine.ansi #122

Closed
remkop opened this issue May 8, 2017 · 0 comments
Closed

API change: remove field CommandLine.ansi #122

remkop opened this issue May 8, 2017 · 0 comments
Labels
type: enhancement ✨
Milestone
0.9.5

Comments

@remkop
Copy link
Owner

remkop commented May 8, 2017

  • Remove public static (non-final) field CommandLine.ansi
  • Make CommandLine.Help.Ansi an enum with values ON, OFF and AUTO
  • Make CommandLine.Help.Ansi.Text a non-static inner class of Ansi, so every Text instance has a reference to its outer Ansi class which detemines whether ANSI escape codes should be emitted or not
  • Add overloaded methods for CommandLine.usage
  • Make it possible to obtain an Ansi reference from the ColorScheme to reduce impact on the Help inner classes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: enhancement ✨
Projects
None yet
Milestone
0.9.5
Development

No branches or pull requests
1 participant
@remkop