-
Notifications
You must be signed in to change notification settings - Fork 563
Add xml comments to the template #251
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| } | ||
|
|
||
| public static int EmptyRegion() | ||
| internal static int EmptyRegion() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
InvalidRegion sounds like a better name than EmptyRegion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's nothing invalid about an empty region?
| ///<param name="package">Takes in the --package option from the code fence options in markdown</param> | ||
| ///<param name="project">Takes in the --project option from the code fence options in markdown</param> | ||
| ///<param name="args">Takes in any additional arguments passed in the code fence options in markdown</param> | ||
| ///<summary>This program uses <a href="https://github.com/dotnet/command-line-api/wiki/DragonFruit-overview">System.CommandLine DragonFruit</a> to accept arguments from command line. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"accept command line arguments"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
System.CommandLine DragonFruit -> System.CommandLine.DragonFruit
No description provided.