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

If config update fails, log to debug instead of warn #793

Closed
ferventcoder opened this Issue Jun 13, 2016 · 0 comments

Comments

Projects
None yet
1 participant
@ferventcoder
Member

ferventcoder commented Jun 13, 2016

If unable to update the config, don't display that - when not running as admin.

@ferventcoder ferventcoder added this to the 0.9.10 milestone Jun 13, 2016

@ferventcoder ferventcoder self-assigned this Jun 16, 2016

@ferventcoder ferventcoder added 3 - Done and removed 0 - Backlog labels Jun 16, 2016

@ferventcoder ferventcoder changed the title from Debug log failure to update config to If config update fails, log to debug instead of warn Jun 16, 2016

ferventcoder added a commit that referenced this issue Jun 17, 2016

(GH-793) Failing to update config - log to debug
Instead of throwing a big red error, it's possible the user doesn't
have access to update the config. So instead of showing a huge warning
and error, log the error to debug output only.

ferventcoder added a commit that referenced this issue Jun 17, 2016

Merge branch 'stable'
* stable:
  (version) 0.9.10
  (GH-398) Default install permissions enhancements
  (doc) update CHANGELOG/nuspec
  (GH-732) reduce default webrequest timeout to 30s
  (maint) fix environmentvariable
  (GH-802) PowerShell Host - Exit codes should return
  (GH-398) Allow append for users on logs directory
  (GH-794) Warn when non-admin in default install
  (GH-793) Failing to update config - log to debug
  (GH-796) Relax upgrade log package separation
  (GH-801) Fix - bad nuspec from choco new
  (legal) add logo use policy
  (GH-795) Adjust message for missing licensed code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment