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
Documented the sexagesimal number parsing "feature" of YAML #11
Conversation
Previous installation URI failed/offline: downloading files from distribution s3 buckets instead. Also, this required new C++ deps that in turn led to a newer base image requirement.
@cblp could you glance over this one and/or merge it, if applicable? |
Oh, I'm sorry! Thank you for poking me! I'll view it asap. |
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.
Reviewed 3 of 3 files at r1.
Reviewable status: complete! all files reviewed, all discussions resolved
Thanks! No need to worry: happens all the time to me as well 👍 |
Documented the sexagesimal number parsing "feature" of YAML (cblp#11)
It sucks harder when you get into longer sexagesimal parsing. eg. mac_address: 52:54:00:44:49:56 For a real world example, see https://tickets.puppetlabs.com/browse/PUP-9505 |
@alexjfisher would you be able to send an example patch? :D |
Ref: https://twitter.com/chrisbranch/status/1094581030147768320
This change is