Results 62 comments of Tommy Bidne

I am interested in this issue, motivated by https://github.com/haskell/cabal/issues/11004. I see `hackage-security` has a [custom Path type](https://github.com/haskell/hackage-security/blob/master/hackage-security/src/Hackage/Security/Util/Path.hs), which would work, though I wonder if there are any libs that would...

Decided to see how this might help out the linked `fourmolu` issue: https://github.com/fourmolu/fourmolu/issues/18 [One-line change](https://github.com/tbidne/fourmolu/compare/main...optparse-groups#diff-d3e498a88b4c374b1edbd8693542a73e944d51a24a64a683658926ba655bc6ffL472) :slightly_smiling_face:: ```diff diff --git a/app/Main.hs b/app/Main.hs index cd53a8a..2a8943f 100644 --- a/app/Main.hs +++ b/app/Main.hs @@ -469,7...

Hi, thanks for the quick response. 1. Re: nested groups. Sure, having the inner group take precedence ("do not overwrite `Just`") is reasonable. I don't personally have any plans for...

**Edit:** I think I figured out what you mean. _Consecutive_ command groups are merged i.e. ```haskell hsubparser ( commandGroup "Group 1" ... ) hsubparser ( commandGroup "Group 1" ... )...

Hopefully this isn't overwhelming, but I pushed two other branches to show what some of discussed changes would look like: 1. [This branch](https://github.com/tbidne/optparse-applicative/compare/parser-groups...parser-groups-new) is the simpler one, which 1) does...

Apologies for the moving target, but I think the conversation has possibly become hard to follow due to separate threads (my fault!), so I've pushed some of the changes to...

Great, thanks! I will likely get to the test updates I mentioned this weekend. Please let me know if there is anything I can do to help.

I made the test changes I mentioned earlier. In particular, I added a test that shows the new parser groups have the same [duplicate semantics](https://github.com/pcapriotti/optparse-applicative/pull/486/files#diff-fb14a94bf75dac5c0631f09768c406ca037de9e6825152a0ee377c931bd81b03) as [command groups](https://github.com/pcapriotti/optparse-applicative/pull/486/files#diff-f7ad077e1e6cd8edff251c6e89bcdb0146f6bb1ba44cf7a3985538d9b57847a2). I also...

Thanks for removing those imports @HuwCampbell (leftovers from a previous patch). Your CI is running into a problem with haskell-ci. See [here](https://github.com/haskell/stm/pull/87) for a workaround and links to more discussion...

Ah I forgot to add the new test to `extra-source-files`. Just pushed up the fix.