Skip to content

Commit a26ae55

Browse files
authored
add translation info to PR template
1 parent 3fdb460 commit a26ae55

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,15 @@
11
Thanks for your interest in plotly.js!
22

3+
### Translations:
4+
5+
- Make a PR directly to the main repository
6+
- Label it `type: translation`
7+
- Please @ mention a few other speakers of this language who can help review your translations.
8+
- If you've omitted any keys from [dist/translation_keys.txt](https://github.com/plotly/plotly.js/blob/master/dist/translation-keys.txt) - which means they will fall back on the US English text - just make a short comment about why in the PR description: the English text works fine in your language, or you would like someone else to help translating those, or whatever the reason.
9+
- You should only update files in `lib/locales/`, not those in `dist/`
10+
11+
### Features, Bug fixes, and others:
12+
313
Developers are strongly encouraged to first make a PR to their own plotly.js fork and ask one of the maintainers to review the modifications there. Once the pull request is deemed satisfactory, the developer will be asked to make a pull request to the main plotly.js repo and may be asked to squash some commits before doing so.
414

515
Before opening a pull request, developer should:

0 commit comments

Comments
 (0)