Skip to content
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

Allow tree to be created even when schema is missing #571

Merged
merged 2 commits into from
Oct 23, 2018

Conversation

drdavella
Copy link
Contributor

Previously an error occurred in the case where a node was tagged with a type from a known extension, but the type itself was unknown. This could happen, for example, in the case where the type is provided by a more recent version of a package, but only the older version of the package is installed.

Now, just a warning occurs, and the affected node is returned as a basic Python tree.

This tests the case where a tag is found from a known, available
extension, but the specific type can't be found. Currently this results
in an error, but ASDF should be able to load files even in this case,
albeit with a warning.
@drdavella
Copy link
Contributor Author

CI hooks are currently busted 😭

@sosey
Copy link

sosey commented Oct 22, 2018

well, at least they did it on purpose 🤷‍♀️ ?

13:32 Eastern Daylight TimeWe have temporarily paused delivery of webhooks while we address an issue. We are working to resume delivery as soon as possible.

@stsci-bot
Copy link

stsci-bot bot commented Oct 22, 2018

Hi there @drdavella 👋 - thanks for the pull request! I'm just a friendly 🤖 that checks for issues related to the changelog. I help make sure that this pull request is milestoned and labeled correctly. This is mainly intended for the maintainers, so if you are not a maintainer you can ignore this, and a maintainer will let you know if any action is required on your part 😃.

Everything looks good from my point of view! 👍

If there are any issues with this message, please report them here.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.009%) to 93.676% when pulling 59afeff on drdavella:schema-unknown-type into a3b9f5d on spacetelescope:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants