Updates ci-build.yml to deploy Commandline tool to Nuget#644
Updates ci-build.yml to deploy Commandline tool to Nuget#644MaggieKimani1 merged 5 commits intovnextfrom
Conversation
.azure-pipelines/ci-build.yml
Outdated
| - task: MSBuild@1 | ||
| displayName: 'Pack OpenAPI.Tool' | ||
| inputs: | ||
| solution: src/Microsoft.OpenApi.Tool/Microsoft.OpenApi.Tool.csproj |
There was a problem hiding this comment.
The NuGet package will be named Microsoft.OpenApi.Tool. Is there a more specific name for this tool?
There was a problem hiding this comment.
From my conversation with @darrelmiller, we're still brainstorming whether to retain it as is or find a more descriptive name for the tool.
There was a problem hiding this comment.
I'm very close to being convinced we should call this Microsoft.hidi as hidi is the Swahili word for Convert.
There was a problem hiding this comment.
Do either of you foresee this tool with future functionality that does something other than convert and validate? If this is foreseeable, then perhaps we leave it as is.
There was a problem hiding this comment.
Yes, it is going to slice. It may also convert from CSDL to OpenAPI. It may in the future be able to apply overlays. I ran the name "hidi" up the chain and we have support from PMs.
There was a problem hiding this comment.
So then, Microsoft.OpenApi.Hidi or Microsoft.Hidi? I'm more for the first, but think both are fine.
There was a problem hiding this comment.
I was planning the second. However, I have been asked to chat with the marketing folks to make sure they are ok with the hidi and kiota names. I'm going try and do that tomorrow.
There was a problem hiding this comment.
We can merge the PR and get approval before we push a release out.
MaggieKimani1
changed the title
3 participants
This PR:
OpenApi.Tool, a commandline tool that enables processing of OpenAPI documents through validation, supports conversion for different file formats e.g fromjsontoyamland vice versa, and also supports filtering/slicing of OpenApi documents into smaller subset documents based on OperationIds and tags provided.Fixes #630