Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We recent encountered some yaml issues. Imagine a yaml file contains an account
number like 012345678. Since it 'seems' to be a number we might expect it to be
quoted in the final yaml output. However, two things cause this to not happen:
a) The number starts with a leading '0' which is interpreted by Python as
octal.
b) Actually, it can not be octal since it contains the digit '8' and so yaml
says that it must be string and discards the quoting because it is not
ambiguous.
This means that in the final accounts.yaml most accounts are quote except for
those that start with a leading '0' and contain either and '8' or a '9'.
This is an adapted fix inspired by the comments in:
https://bitbucket.org/xi/pyyaml/issues/29/zero-padded-numbers-ending-in-8-or-9-dump