Skip to content
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

add continents #32

Merged
merged 6 commits into from
Feb 13, 2013
Merged

add continents #32

merged 6 commits into from
Feb 13, 2013

Conversation

lukkry
Copy link
Contributor

@lukkry lukkry commented Oct 20, 2012

I've written tiny script to assign continents. It uses this gem to get continents' names.

I've assigned all countries from Oceania region to Australia continent. It could be great if someone can confirm if it's correct ;)

@hexorx
Copy link
Collaborator

hexorx commented Oct 22, 2012

You are on the right path but according to (https://github.com/ravibhim/continents/blob/master/lib/ravibhim/data/australia.yml) the only two countries considered part of the "Australia" continent are Australia and New Zealand. Maybe your script could be modified to just open each of the yml files in that gem and build up a list of country codes for each continent. I think some island countries might not be part of a country just like Antarctica is a continent with no countries.

@lukkry
Copy link
Contributor Author

lukkry commented Feb 9, 2013

I've completely changed my approach and rewrited the script. This time I use MaxMind's data to obtain continents' names. I believe this data is reliable.

Any thoughts?

@hexorx
Copy link
Collaborator

hexorx commented Feb 9, 2013

That data looks good to me.

Sent from Mailbox for iPhone

On Sat, Feb 9, 2013 at 4:51 PM, Lukasz Krystkowiak
notifications@github.com wrote:

I've completely changed my approach and rewrited the script. This time I use MaxMind's data to obtain continents' names. I believe this data is reliable.

Any thoughts?

Reply to this email directly or view it on GitHub:
#32 (comment)

hexorx added a commit that referenced this pull request Feb 13, 2013
@hexorx hexorx merged commit 7b67a05 into countries:master Feb 13, 2013
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.

2 participants