feat: append korean names

[i-am-a-dolphin]

Ref: https://yiunsr.tistory.com/885

[netlify]

✅ Deploy Preview for fakerjs ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	c16d76b
🔍 Latest deploy log	https://app.netlify.com/projects/fakerjs/deploys/69915784328d520008c49569
😎 Deploy Preview	https://deploy-preview-3742.fakerjs.dev
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[ST-DDT]

Hi @i-am-a-dolphin ,

Faker usually limits lists of names to ~1k entries to save memory/disk space a bit.
Is there a reason why you would add all names you have obtained prgrammatically?
(if the current set needs to be refreshed, then thats a valid reason, we should just stick to ~1k names)

Also, have you checked whether that particular source may be subject to copyright/ownership issues?
(the names itself aren't copyright-able, but specific collections might be without own transformation)

[ST-DDT]

@i-am-a-dolphin Are you still interested in this PR?

[ST-DDT]

This PR appears to have gotten stale. If you are stil interested in this please let us know and we can reopen this one.