Skip to content

Comments

Remove symbol keys#39

Open
konklone wants to merge 1 commit intocarr:masterfrom
sunlightlabs:master
Open

Remove symbol keys#39
konklone wants to merge 1 commit intocarr:masterfrom
sunlightlabs:master

Conversation

@konklone
Copy link

This removes the symbols from keys in phone_countries.yml, and changes the load process to expect string keys instead.

This makes the phone gem compatible with applications using safe_yaml, or other means of sanitizing the YAML loading process to avoid Ruby-specific extensions.

Fixes #38.

@elskwid
Copy link
Collaborator

elskwid commented May 9, 2013

I'll be looking this over soon. Thank you for your patience. See #41.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Phone doesn't work with safe_yaml

2 participants