Move generator shared source files to Azure.Core shared source#38286
Move generator shared source files to Azure.Core shared source#38286annelo-msft merged 3 commits intoAzure:mainfrom
Conversation
|
API change check API changes are not detected in this pull request. |
christothes
left a comment
There was a problem hiding this comment.
Should we log an issue to close some of the test coverage gaps here?
@christothes, which test coverage gaps? |
There may not be any in practice, I was just comparing classes with test classes and there appear to be some missing tests. |
Cool, good call out, then. I'll file that issue - do you have a list handy, or is it primarily just "there are methods without tests validating their functionality?" |
The latter |
|
I think this is ready to go now, but will wait for @m-nash for final approval to make sure I'm not missing something in all the back-and-forth steps across repos. Next steps are:
|
Fixes #32006
Azure/autorest.csharp#3674 and #38305 show proof of concept that once Azure.Core shared source in the autorest.charp repo is updated by the script, we expect build and test to succeed.