-
Notifications
You must be signed in to change notification settings - Fork 143
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
testing: add an integration test for yaml-test-suite #187
base: master
Are you sure you want to change the base?
Conversation
The official YAML test suite (https://github.com/yaml/yaml-test-suite). Requires the submodule to be checked out.
I would like to mention that I added yaml-rust to the test matrix: https://matrix.yaml.info/ with the help of @tanriol |
Well, according to @chyh1990's profile, their last activity was on June 3, 2020. But even before that, the activity was rather sparse. PRs and issues just kept accumulating without any meaningful progress for years now. Maybe there are more active forks already? |
I am not involved in this crate, since it is no longer used by serde_yaml. |
Sounds reasonable given the maintenance state of this crate. I'm leaving this PR open just in case, but don't really care about its fate in this case. |
FWIW I have a fork I'm maintaining since I need this to keep working for the long-haul: https://github.com/davvid/yaml-rust/ I'd be happy to take PRs related to this topic there. |
The official YAML test suite (https://github.com/yaml/yaml-test-suite).
Requires the submodule to be checked out.
This is a draft PR because there's quite a number of failures. At the moment I'm seeing 62 unexpected errors (and 19 more errors where
yaml-rust
Event
/Parser
/EventReceiver
API does not provide enough information about the document). I can mark the other errors as expected errors for now, but want to look at them first to check whether they'reyaml-rust
errors or some mistakes in this test harness.