-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
validate example image resources #95
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems okay, although I'm not a fan of code duplication 😛 but, since we are doing it already, probably that's okay for now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this reminds me we need to drop schematics and standardize on pydantic. it'll block us from upgrading since it's no longer maintained. (unless we switch to schematics/schematics#595)
return answer_uris | ||
|
||
|
||
def validate_requester_restricted_answer_set_uris(restricted_answer_set: dict) -> None: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for remembering about this one!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, but the error message can be a bit misleading
basemodels/pydantic/manifest/data/requester_question_example.py
Outdated
Show resolved
Hide resolved
created a jira ticket for this |
No description provided.