azure-search-openai-demo
azure-search-openai-demo copied to clipboard
Keep the Approach base class up to date
Purpose
Approach base class was not up to date with the newer run() schemas.
Using abc, we get more confidence that all subclasses conform with the declared schema.
Does this introduce a breaking change?
[ ] Yes
[x] No
Pull Request Type
What kind of change does this Pull Request introduce?
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:
@tonybaloney done!
@gukoff Thanks for this PR, I ended up making the same changes in a recent PR that changed the interface, so we don't need to merge this anymore.