-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Avoid jargon and slang, change for default, de and pl (#2874) #2876
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
Avoid jargon and slang, change for default, de and pl (#2874) #2876
Conversation
I don't think we should be editing the non-default language strings directly, everything except default will be overriden by translatewiki's automatic updates AFAIK. For the default, I'll leave a comment on the code itself. |
Ok, so i close this pull request. |
@marcin-kwasniak , you can make changes within the PR itself. If you modify your code, commit it to the same branch and push the branch, we will automatically get your changes. |
Codecov Report
@@ Coverage Diff @@
## master #2876 +/- ##
=========================================
- Coverage 3.42% 3.42% -0.01%
=========================================
Files 246 246
Lines 12193 12208 +15
Branches 1071 1072 +1
=========================================
Hits 418 418
- Misses 11744 11759 +15
Partials 31 31
Continue to review full report at Codecov.
|
@misaochan I made change, is it ok now? |
Hi @marcin-kwasniak , My suggestion was to have "Category check successfully requested" for the success string, because "category check requested" does not necessarily mean success IMO. Your text for the failure string was fine, there was no need to remove it. |
My mistake, new text for fail is commited. |
Thanks, and congrats on your first PR! :) |
"check category failure title" and "check category success title" shouldn't use the words "success" and "failure"
Fixes #{#2874} for default, de and pl