command-line-api icon indicating copy to clipboard operation
command-line-api copied to clipboard

Create error reporting output

Open KathleenDollard opened this issue 1 year ago • 2 comments

This will be a good first issue for someone who has some understanding of System.CommandLine and wants to understand how the new pipeline design works. It involves finding the error reporting code in System.CommandLine extended (in main-powderhouse branch and putting it in ErrorReportingSubsystem.Execute

This part is very easy. However, testing will require resolution of #2345.

If you do not want to wait, consider:

  • Use an internal method in ErrorReportingSubsystem that takes a collection to avoid the ReadonlyList on ParseResult
  • Copy 'ParseError' to the System.CommandLine.Subsystem (I think that gives it precedence) or copy it and give it a recognizable fake name like ParseError2
  • Do a draft PR because we can't take this as is. The draft PR will ensure we move the decision in #2345 to the front of the queue of things to resolve

KathleenDollard avatar Mar 07 '24 13:03 KathleenDollard

Taking a look at this, will put up a draft PR, but it sounds like #2345 is leaning towards ParseError being public so I'll write with that in mind

JeanJoeris avatar Mar 18 '24 16:03 JeanJoeris

Would love to contribute!

karshav16 avatar Oct 04 '24 09:10 karshav16