Add validator to catch datetime azure_openai.version #958
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🤔 Why?
If the
azure_openai.version
set in the app is a regular version e.g.2024-06-01
instead of a-preview
version it can be misinterpreted as a datetime when the config is loaded, causing validation errors.🤓 What?
Adds a validator to force datetime to string to catch pesky validation errors.
🧪 Tested?
Connector tested locally with a YAML file generated by
app
, runs successfuly.☑️ Checks
pyproject.toml
.