8000 Fix CCA2 and alternative spellings for Norway by sensen · Pull Request #96 · mledoze/countries · GitHub
[go: up one dir, main page]

Skip to content

Fix CCA2 and alternative spellings for Norway #96

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

Closed
wants to merge 1 commit into from
Closed

Fix CCA2 and alternative spellings for Norway #96

wants to merge 1 commit into from

Conversation

sensen
Copy link
@sensen sensen commented Dec 31, 2014

NO is reserved keyword in YAML. It'll become false when loaded instead
of literal NO.

http://yaml.org/type/bool.html

NO is reserved keyword in YAML. It'll become `false` when loaded instead
of literal NO.

http://yaml.org/type/bool.html
@petert82
Copy link
Contributor
petert82 commented Jan 2, 2015

This is going to be a bit of a pain to fix properly. The Symfony YAML component that is being used to dump the country data to YAML 'mostly' implements YAML 1.2 where boolean values can only be represented as true or false. It doesn't quote strings that would be interpreted as boolean values ('yes', 'no', 'on', 'off', etc) by parsers implementing older versions of the YAML spec.

I'm going to raise an issue with the Symfony Yaml component to ask if they would consider a patch to quote these values when dumping to YAML.

@sensen
Copy link
Author
sensen commented Jan 4, 2015

@herrniemand I wasn't aware of this. As @petert82 said, the proper way to fix this issue is by fixing the Symfony YAML component.

@mledoze
Copy link
Owner
mledoze commented Jan 7, 2015

@sensen thank you for raising this issue, I'll close it for now and wait for a fix on the Symfony YAML component.

@mledoze mledoze closed this Jan 7, 2015
@petert82
Copy link
Contributor
petert82 commented Jan 7, 2015

Just for reference, this should be fixed once this PR is accepted. I'll make sure to keep an eye on it and update the composer.json here, if necessary once this is done.

@mledoze
Copy link
Owner
mledoze commented Jan 7, 2015

@petert82 very good, thank you! I'll keep an eye on it too.

@shehi
Copy link
shehi commented Jan 7, 2015

@mledoze I strongly recommend against closing such issues - why don't you wait with it open and not miss the issue when Symphony fixes things at their side?

@mledoze
Copy link
Owner
mledoze commented Jan 7, 2015

@shehi you're right, no problem.

@mledoze mledoze reopened this Jan 7, 2015
@mledoze mledoze closed this in a3c480d Feb 5, 2015
@mledoze
Copy link
Owner
mledoze commented Feb 5, 2015

The pull request symfony/symfony#13262 has been merged into v2.6.4. I updated the dependencies in a3c480d.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0