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

country_converter/country_data.tsv: fix UN M.49 region #123

Merged
merged 1 commit into from
Nov 14, 2022

Conversation

alanorth
Copy link
Contributor

@alanorth alanorth commented Nov 7, 2022

According to UN M.49 the region is "South-eastern Asia" with the word "eastern" in lower case. You can see this on the website as well as the CSV data export.

See: https://unstats.un.org/unsd/methodology/m49/#geo-regions
See: https://unstats.un.org/unsd/methodology/m49/overview/

Fixes #115

According to UN M.49 the region is "South-eastern Asia" with the word
"eastern" in lower case. You can see this on the website as well as the
CSV data export.

See: https://unstats.un.org/unsd/methodology/m49/#geo-regions
See: https://unstats.un.org/unsd/methodology/m49/overview/
alanorth added a commit to ilri/csv-metadata-quality that referenced this pull request Nov 8, 2022
Until a few issues are resolved regarding new countries and regions.

See: IndEcol/country_converter#122
See: IndEcol/country_converter#123
@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.03% when pulling d4f4b13 on alanorth:se-asia-un-m49 into eda3a0f on konstantinstadler:master.

@konstantinstadler
Copy link
Member

Tanks a lot!

@konstantinstadler konstantinstadler merged commit 14d419f into IndEcol:master Nov 14, 2022
@alanorth alanorth deleted the se-asia-un-m49 branch November 14, 2022 13:38
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.

Case issue in UN M.49 region "South-eastern Asia"
3 participants