redo demo for authoring

FirelyTeam / firely-cql-sdk

BSD 3-Clause "New" or "Revised" License

26 stars 15 forks source link

Closed richfirely closed 1 month ago

richfirely commented 1 month ago

Merged all the great code from CLI.CMS to CLI and retired CLI.CMS
Added Measures.Authoring project for creating new CQL measures.
Made all the measures projects exe's for simplicity
Removed CLI direct dependency on Measures projects. Goes the other way around.
Retired graph logic and loop through resources directly
Added valueset checking to CLI
Added test data and related input directories to align with vscode tooling. More...

baseTwo commented 1 month ago

CLI input argument validations should not be displayed as an exception to the end-user. (perhaps another issue for later if you are pressed for time)

baseTwo commented 1 month ago

Runtime exceptions when running CLI:

BCSE profile: System.ArgumentException: Please choose one: either assemblyPath or resourcesDirectory. StackTrace: CommandLineOptions.cs:line 62
AdultAO_FromResources and AdultAO_FromProject profiles: System.ArgumentException: The value (options.OutputFile) cannot be an empty string. LibraryRunner 31

I suspect it's because the argument names and values that can be provided in the new CommandLineOptions have not been applied to launchSettings.json

richfirely commented 1 month ago

CLI input argument validations should not be displayed as an exception to the end-user. (perhaps another issue for later if you are pressed for time)

Thanks! I am not clear on what you mean by this one. I will fix the rest and merge it for now and revisit that when you are in the office.