datasets/country-codes

GB (UK) & USD lost currencies

Closed this issue · 4 comments

wodow commented

Currency codes for GB & US, at least, were lost in the jumbo commit d4e4895.

Too long to check all the edits, so maybe revert?

Thanks for catching this, @wodow

There was a bug in the currency code script that was swallowing errors, so several currencies were being excluded silently.
Have fixed GB, US, and a few more!

wodow commented

Thank you, @ewheeler

I think unit tests or simple assertions on the output would help. (I could try taking a look at that later in the month.)

you're welcome, @wodow!

yes i agree that simple tests would help to avoid regressions.

i've created a separate issue to track here: #40
would welcome your thoughts and ideas!

cheers

Can it be the reason taiwan is missing all its currency info?

d4e4895#diff-c1f0a2a818473ec463559fafe9ce27f5L218