Ryan Leckey

Results 206 comments of Ryan Leckey

I'd agree documentation could be improved here. --- I'm not sure how to support "merging" arrays and still allow "replacing" arrays (which I consider more common). Perhaps a method that...

Thought about this recently. I'd like to be able to do this: ```rust config.set_array_merge_strategy(MergeStrategy::Replace); // Default // --- enum MergeStrategy { Replace, Append, AppendUnique, } ``` I agree that per-key...

```rust config::merge(config::File::with_name("Settings").weight(10)); config::merge(config::Environment::with_prefix("rust").weight(5)); ``` That feels fine. Interesting suggestion. I'll peg it to be added.

Thinking more on this .. I don't like `.weight` being on the source API 1. `Config.merge_at` takes a second, `weight`, parameter ```rust config.merge_at(File::with_name("Settings"), 10); ``` 2. Complicate the return type...

SQLx is going to push out support for May soon, tracking here: https://github.com/launchbadge/sqlx/issues/220

@b4b4r07 @agross I feel that `defer`, `nice`, etc. are being far over thought. Let's make this simple. There are two distinct concepts that matter: - Module should be loaded _after_...

For some prior art, I've always loved how C# does this: ```csharp [StructLayout(LayoutKind.Explicit, Size = 4)] struct Foo { [FieldOffset(0)] public byte bar; [FieldOffset(0)] public int baz; } ``` To...

@andrewbanchich @muppetjones Is the ui in `linter-ui-plus` an improvement? I strongly feel that the aversion to `linter-ui-default` is not that they are icons but that they are _line_ icons.

@andrewbanchich Honestly, I disagree with you. But let's stop and appreciate that Atom is a hackable editor. If you use `linter-ui-plus` you can apply the following in your `styles.less` to...

Thanks @Positive07 , I dropped that pack into my linter ui experiment. #352 https://github.com/mehcode/linter-ui-plus