-
Notifications
You must be signed in to change notification settings - Fork 57
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
added extraction of title from schema to show as a comment in info() #1138
added extraction of title from schema to show as a comment in info() #1138
Conversation
for more information, see https://pre-commit.ci
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.
LGTM
@@ -53,7 +53,7 @@ docs = | |||
tests = | |||
# Necessary for Windows until https://github.com/pytest-dev/pytest/issues/9765 | |||
# is resolved: | |||
pytest!=7.1.0, !=7.1.1;sys_platform=="win32" | |||
pytest!=7.1.0, !=7.1.1, !=7.1.2;sys_platform=="win32" |
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.
I am trying to track a failure in Jdaviz and I am suspecting it has something to do with asdf
. Why did you pin a !=7.1.2
here?
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.
Oh hmm, this only is for Windows... my failure is on Linux.
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.
Does this error ring any bell to you?
2022-05-25T20:01:04.0755535Z INTERNALERROR> Traceback (most recent call last):
2022-05-25T20:01:04.0756248Z INTERNALERROR> File "<frozen importlib._bootstrap>", line 939, in _find_spec
2022-05-25T20:01:04.0756783Z INTERNALERROR> AttributeError: '_SixMetaPathImporter' object has no attribute 'find_spec'
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.
Is this for ASDF?
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.
Not sure. It looks like it is coming from one of the plugins and you ship with your own pytest plugin, right? I'll dig more.
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.
Sorry for the noise. It's not ASDF. Something happened in the Python 3.10 that shipped with Actions runner.
Note that due to how the extension manager works, for the test file this required adding an option to info to force reloading of the extension manager so that the test extensions would be seen. There may be a better way to handle this. It should handle attributes that use patternProperty, oneOf, allOf, or anyOf constructs. This addresses issue #1137.