openfoodfacts / openfoodfacts-server

Open Food Facts database, API server and web interface - 🐪🦋 Perl, CSS and JS coders welcome 😊 For helping in Python, see Robotoff or taxonomy-editor
http://openfoodfacts.github.io/openfoodfacts-server/
GNU Affero General Public License v3.0
647 stars 374 forks source link

Remove final /n in Crowdin translations #6591

Open teolemon opened 2 years ago

teolemon commented 2 years ago

What

Logs

po/common/lol.po:145: 'msgid' and 'msgstr' entries do not both end with '\n' po/common/lol.po:234: 'msgid' and 'msgstr' entries do not both end with '\n' po/common/lol.po:794: 'msgid' and 'msgstr' entries do not both end with '\n' po/common/lol.po:1213: 'msgid' and 'msgstr' entries do not both end with '\n' po/common/lol.po:1541: 'msgid' and 'msgstr' entries do not both end with '\n' po/common/lol.po:1946: 'msgid' and 'msgstr' entries do not both end with '\n' msgfmt: found 6 fatal errors → po/common/lol.po...contains errors! → po/common/nb.po...ok → po/common/he.po...ok → po/common/mr.po...ok → po/common/oc.po...ok → po/common/qu.po...ok → po/common/km.po...ok → po/common/lt.po...ok → po/common/gl.po...ok → po/common/ca.po...ok → po/common/sv.po...ok → po/common/et.po...ok → po/common/ha.po...ok → po/common/zu.po...ok → po/common/ce.po...ok → po/common/lv.po...ok → po/common/ia.po...ok po/common/uk.po:1140: 'msgid' and 'msgstr' entries do not both end with '\n' po/common/uk.po:4475: 'msgid' and 'msgstr' entries do not both end with '\n'

teolemon commented 2 years ago

actually in source strings @stephanegigandet

alexgarel commented 2 years ago

@teolemon I don't understand. The checker is about msgid ending with \n while msgstr do not end with one (or the other way around). It's more a quality warning than a real error. (and note that in HTML \n will be ignored so it's not a real problem)

  1. in lol.po this are crowdin messages ids (I imagine this is used by crowdin to track known messages), so you just have to ignore that file, it will always be wrong respect to that.
  2. in uk.po, this is the translator who adds the '\n', so it must be fixed in crowdin.
github-actions[bot] commented 2 years ago

This issue is stale because it has been open 90 days with no activity.