Adrian Sampson

Results 1057 comments of Adrian Sampson

I think the main thing to realize is that the line `config.get(template)` isn't doing anything—it doesn't change anything about the configuration. `get` is really just a getter and won't affect...

Great idea! This might be hard to support in some backends, like the ffmpeg one, but that shouldn't stop us from adding it to backends where it's easy/possible. One thing...

Interesting use case! Indeed, this will not work yet. To clarify, our current round-tripping of comments is quite hacky, but it could plausibly be greatly improved by switching to ruamel.yaml...

Indeed—that's the tricky thing here. When you do `config["active_labels"].set([2, 3])`, you are actually not modifying any configuration data structure in place! Confuse works by "layering" together multiple configurations and then...

Well, here's the thing: you can actually dump a "changed" version of the config file, but the way that "change" works is pretty subtle. You're not actually modifying one data...

I don't currently have plans to implement it, so any help would be greatly appreciated! However, part of the point of the discussion above was to emphasize plain round-tripping from...

Awesome; thanks for getting this started! One other thing that #52 mentions is the OrderedDict extension. You can see where we preserve dictionary order here: https://github.com/beetbox/confuse/blob/c328e810f7a31412e0650235f71728e983edd18e/confuse/yaml_util.py#L31 It might be nice...

The first thing I might try is checking whether this happens on a specific file. It might not be supported by any of the backends you have.

No, sorry! To get help from other people, it's super-duper important to be able to narrow down the problem to something that can be reliably reproduced on someone else's machine....

Thanks for pointing that out; it does seem like we should invert this. One thing to be careful about when making the switch is that this prioritization also affects the...