Kevin Chowski

When parsing a 38-char UUID, require '{' and '}' characters

10

It was not intentional to simply ignore these characters, and examples/docs/tests all indicate that curly braces were the intended characters. I've added some test cases to ensure things work. Fixes...

Parsing "Microsoft encoding" is very lenient

2

I was reading through the code for Parse and I noticed that there is no validation on the first and last characters of the uuid-to-parse when the length of the...

Naming clash for new cmp standard package

4

I am not sure where the best place to have this discussion is, but I wanted to get @dsnet's opinion on this so this seemed like a reasonable place to...

Is there any more information about the __rseq_table convention available?

2

Looking through various mailing lists (and code in this repo) there are references to an __rseq_table section which would ideally contain useful metadata about the restartable sequences in the program....

Subcommands should have first-class support for errors

2

Currently, the [CommandRun.Run](https://pkg.go.dev/github.com/maruel/subcommands#CommandRun) method returns an int. I suspect this is because the original design wanted commands to decide their exit code, but it causes some unfortunate boilerplate: most commands...