-
Notifications
You must be signed in to change notification settings - Fork 520
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
on
is loaded as True instead of a string
#376
Comments
It is actually correct. If you want I'm not sure what the problem has to do with inline lists. As far as I can see you habve a problem with Feel free to reopen if you have more questions. |
on
is loaded as True instead of a string
Thanks for your detailed explanation, I understand how to do, This issue was raised because some other implementations did not resolve In addition, I'm sorry, the title was not described very accurately, because I didn't know these details before, I naturally searched for the same keywords, and then raised this issue. If that can better help others who have the same as me Confused people, that couldn't be better. |
@StrayDragon Actually you can change this behavior if you want by overriding the bool tag regex in the base resolver (https://github.com/yaml/pyyaml/blob/master/lib/yaml/resolver.py#L170-L175), then wire up your resolver instead of the default on a new loader instance (which gets passed to |
@nitzmahone Thanks for your reply, now, I know how to change this behavior at least. |
This comment was marked as outdated.
This comment was marked as outdated.
Is this still correct when |
Yes, this is correct. I recently pulished yamlcore which you can use on top of PyYAML. It will treat github workflow files correctly. |
Minimum reproducible situation, complete code here
Use PyYaml 5.3 and CPython 3.8.1
will output:
Obviously, it's a wrong answer... and expect
{'on': ['push']}
The text was updated successfully, but these errors were encountered: