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.

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

Moving to CommandLineUtils #324

Merged
merged 13 commits into from Jul 21, 2018

Conversation

Projects
None yet
2 participants
@skolima
Collaborator

skolima commented Jul 10, 2018

Exploring options for #323

Status:

  • inspect starts
  • inspect accepts options and help
  • logging levels can be set
  • update command works
  • repository command works
  • organisation command works
  • TODO: various smaller configuration settings
  • documentation update
  • version output correct

This would not include configuration file support.

@skolima

This comment has been minimized.

Collaborator

skolima commented Jul 10, 2018

NuKeeper --help
image

NuKeeper inspect --help
image

@AnthonySteele

This comment has been minimized.

Member

AnthonySteele commented Jul 11, 2018

Allowed change should not show the None member of that enum.

@skolima

This comment has been minimized.

Collaborator

skolima commented Jul 12, 2018

Verbosity settings would now look like this (syntax snatched from dotnet):

image

@AnthonySteele could you have a look later at e44817a and tell me what you think? I had to make the logger mutable, but that is hidden by an explicit interface implementation.

@AnthonySteele

This comment has been minimized.

Member

AnthonySteele commented Jul 21, 2018

Ready to merge?

@skolima

This comment has been minimized.

Collaborator

skolima commented Jul 21, 2018

I'd say it's ready now. I'll add the "TODO" checklist on the Issue itself.

@skolima skolima removed the DoNotMergeYet label Jul 21, 2018

@AnthonySteele AnthonySteele merged commit 290b01a into NuKeeperDotNet:master Jul 21, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@skolima skolima deleted the skolima:commandline branch Jul 23, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment