-
Notifications
You must be signed in to change notification settings - Fork 270
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
Crashes on encountering emojis #48
Comments
Thanks for reporting this! I think it's related to yaml/pyyaml#25 I'm curious about where you found emojis in YAML? |
Writing cloudformation templates and someone tried to add 🤷♂️ in a comment. |
The fix in PyYAML has been merged, but they dont release yet, and havent for a long time. A bit of discussion regarding releasing at yaml/pyyaml#46 Maybe switch to using |
This is unfortunately still an issue on 1.15.0, are there any plans to fix this? Happy to submit a PR switching to ruamel.yaml if that's of use? |
I have same issue. Is there any solution? |
PyYAML 5.1 was just released, so the bug should be fixed now (by yaml/pyyaml#63). Could you test with this new version? |
My reported test case now passes, yay! Thanks for the update @adrienverge :) |
I'm glad to hear it 🙂 |
Apparently PyYAML just crashes when loading emoji, but it's not clear why:
If there's some flag inside PyYAML that can be used to allow them, I think
yamllint
should activate it to allow it to lint the widest possible range of yaml files.The text was updated successfully, but these errors were encountered: