-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add dotnet tool ability to XMLtoMD proj #157
Conversation
BenjaminMichaelis
commented
Mar 6, 2021
- DragonFruit CLI tool added
- pack and nupkg tool ability added
- upgraded from .netcore3.1 to .net5.0 - suppressed warning syslib0012 Will need to address https://docs.microsoft.com/en-us/dotnet/core/compatibility/syslib-warnings/syslib0012 Most likely by using Assembly.Location
- added XML nodes for ability to pack and distribute the application as a nupkg
.Net 5.0 upgrade will be coming in the future
XMLtoMD/GuidelineXmlToMD/Program.cs
Outdated
/// <param name="i">The xml file to process when using the -m option</param> | ||
/// <param name="o">The md file to create when using the -m option </param> | ||
/// <param name="m">Manual Run - Use xmlFilePath and outputMDFilePath</param> | ||
static void Main(string[] args, string xmlFileName, string i = "null", string o = "null", bool m = false) |
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.
You shouldn't need to take in the string args and instead just specify the parameters that you need.
This will likely require changing the invocation on the GitHub Action.
You can use FileInfo
as the parameter type and it will properly parse it for you.
static void Main(string[] args, string xmlFileName, string i = "null", string o = "null", bool m = false) | |
static void Main(string xmlFileName, string i = "null", string o = "null", bool m = false) |
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.
Removed default args []
Co-authored-by: Kevin B <Keboo@users.noreply.github.com>
What's the status on this? @Keboo |