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

qri config set unable to set logging levels #450

Closed
ramfox opened this issue Jun 7, 2018 · 1 comment · Fixed by #459
Closed

qri config set unable to set logging levels #450

ramfox opened this issue Jun 7, 2018 · 1 comment · Fixed by #459
Assignees
@b5
Labels
bug code that is not behaving as expected
Milestone
0.5.0

Comments

@ramfox
Copy link
Member

ramfox commented Jun 7, 2018 

$ qri config set logging.levels.qrip2p debug
reflect: reflect.Value.SetString using unaddressable value
@ramfox ramfox added the bug code that is not behaving as expected label Jun 7, 2018
@ramfox
Copy link
Member Author

ramfox commented Jun 7, 2018 •
edited
Loading 

$ qri config set logging.levels {"qrip2p":"debug"}
config updated

The above works returns with config updated, however if you qri config get, no change was made.

@b5 b5 added this to the 0.4.1 milestone Jun 8, 2018
@b5 b5 added the ready label Jun 11, 2018
b5 added a commit that referenced this issue Jun 12, 2018
@b5
fix(config): set map values with config set command
edd9e44 
updated config Set method to support setting map[string]interface{} values

closes #450
@b5 b5 mentioned this issue Jun 12, 2018
Merged
@ghost ghost assigned b5 Jun 12, 2018
@ghost ghost added in progress and removed ready labels Jun 12, 2018
@b5 b5 closed this as completed in #459 Jun 12, 2018
@ghost ghost removed the in progress label Jun 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@b5 b5

Labels
bug code that is not behaving as expected
Projects
None yet
Milestone
0.5.0
Development

Successfully merging a pull request may close this issue.

fix(config): set map values with config set command qri-io/qri
2 participants
@b5 @ramfox