Dan Buch

Results 145 comments of Dan Buch

Looking at this again, I think the change to automatically sorting the flags is arguably a regression because the current documentation claims that the default ordering is *not* sorted.: https://cli.urfave.org/v2/#ordering

@binwiederhier Hello and sorry for the delay! 😭 This looks like a welcome change to me. Are you up for proposing a PR?

@klueska sorry for my long delay! Given some recent related work by @joeycumines at the top-level package, I'd love to see if there's a solution to `altsrc/` that can follow...

@joeycumines I hope I'm understanding you correctly, but making `altsrc` lower precedence than the default defined in code would be different ordering than is claimed in the documentation: https://cli.urfave.org/v2/#precedence In...

> My bad, when I said > > > Even assuming altsrc is the lowest priority, this seems like it would effectively break default values. > > I wasn't including...

@dearchap hello! Sorry, I should have said something about _why_ I marked this as "Draft" πŸ˜… I'm not expecting feedback yet, because I'm not convinced this is working at all...

Nope! I'll revisit in the future if it seems like a reasonable approach.

@ipostelnik Hello and sorry for the delay and auto-close via stalebot. Are you up for revisiting this work, resolving conflicts, and incorporating changes from feedback?

@asahasrabuddhe I'm interesting in continuing this work if you're not able βœ‹πŸΌ