Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Create Crowdin Glossary #81

Closed
zeke opened this issue May 8, 2018 · 1 comment · Fixed by #91
Closed

Create Crowdin Glossary #81

zeke opened this issue May 8, 2018 · 1 comment · Fixed by #91

Comments

@zeke
Copy link
Contributor

zeke commented May 8, 2018

Crowdin has a glossary feature that presents translators with useful information about the terms they're translating (or should not be translating). Here's an example:

36569408-8a4ad454-17e2-11e8-8a5b-9c394db0eafd

For nodejs/i18n, we should probably start with the following:

  • node module names: fs, path, net, etc
  • JavaScript builtins like Array, Boolean, etc. We can use the builtins module for this.

I've extracted our Crowdin glossary code from electron/i18n and created a new node module crowdin-glossary that we can use:

const glossary = require('crowdin-glossary')({
  project: 'nodejs'
})

Object.keys(globals.builtin).forEach(term => {
  glossary.add(term, 'This is a JavaScript builtin and should usually not be translated.')
})
 
glossary.upload()

For more info on Electron's glossaries, see https://github.com/electron/i18n/blob/master/contributing.md#glossaries

@lukaszewczak
Copy link
Contributor

Hi @zeke , I can prepare PR with such a script for nodejs/i18n

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants