[vim/vim] Update some Spanish msgid strings (#8900)

12 views
Skip to first unread message

cquike

unread,
Sep 21, 2021, 5:09:35 AM9/21/21
to vim/vim, Subscribed

Some of the msgid strings in es.po where outdated and I have now bring them up to date.
Note that there might be other outdated msgid. The ones I have modified where found while using vi and realizing there are untranslated messages. Probably there are other msgid strings that are yet to be synchronized.
I have tried the compiles es.mo and it now shows the proper message in Spanish.


You can view, comment on, or merge this pull request online at:

  https://github.com/vim/vim/pull/8900

Commit Summary

File Changes

Patch Links:


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Bram Moolenaar

unread,
Sep 21, 2021, 2:04:23 PM9/21/21
to vim/vim, Subscribed

Thanks. The file has not been updated in a long time. Did you try to contact the maintainer? I can include this patch, but it would be good to update all the translations.

Tarunav.BA

unread,
Sep 23, 2021, 5:18:51 AM9/23/21
to vim/vim, Subscribed

@cquike looks good.

cquike

unread,
Sep 23, 2021, 8:41:46 AM9/23/21
to vim/vim, Subscribed

I contacted the maintainer after creating the PR actually, but has not replied yet.
Indeed there are more translations to update. I might go back to those when I have time. BTW, is there a standard workflow for the translators to get updated msgid strings?

Christian Brabandt

unread,
Sep 23, 2021, 8:45:19 AM9/23/21
to vim/vim, Subscribed

cquike

unread,
Sep 23, 2021, 9:30:03 AM9/23/21
to vim/vim, Push

@cquike pushed 1 commit.

  • c8d033b Update some Spanish msgid strings


You are receiving this because you are subscribed to this thread.

View it on GitHub.

cquike

unread,
Sep 23, 2021, 9:38:03 AM9/23/21
to vim/vim, Subscribed

Thank you @chrisbra. I should have guessed it :) I think it is going to be a bit of a daunting task to update the es.po file...
I have updated the commit to fix the CI problem in freebsd_12. Apparently the headers of the po file didn't contain a Plural-Form keyword.

codecov[bot]

unread,
Sep 23, 2021, 9:20:45 PM9/23/21
to vim/vim, Subscribed

Codecov Report

Merging #8900 (c8d033b) into master (770ddbe) will decrease coverage by 87.72%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@

##           master    #8900       +/-   ##

===========================================

- Coverage   90.18%    2.45%   -87.73%     

===========================================

  Files         151      149        -2     

  Lines      170772   165893     -4879     

===========================================

- Hits       154015     4077   -149938     

- Misses      16757   161816   +145059     
Flag Coverage Δ
huge-clang-none ?
huge-gcc-none ?
huge-gcc-testgui ?
huge-gcc-unittests 2.45% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/float.c 0.00% <0.00%> (-99.22%) ⬇️
src/gui_gtk_f.c 0.00% <0.00%> (-97.54%) ⬇️
src/sound.c 0.00% <0.00%> (-97.12%) ⬇️
src/crypt_zip.c 0.00% <0.00%> (-97.06%) ⬇️
src/cmdhist.c 0.00% <0.00%> (-97.03%) ⬇️
src/match.c 0.00% <0.00%> (-96.98%) ⬇️
src/sha256.c 0.00% <0.00%> (-96.94%) ⬇️
src/evalbuffer.c 0.00% <0.00%> (-96.92%) ⬇️
src/textprop.c 0.00% <0.00%> (-96.85%) ⬇️
src/libvterm/src/rect.h 0.00% <0.00%> (-96.56%) ⬇️
... and 137 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 770ddbe...c8d033b. Read the comment docs.

Bram Moolenaar

unread,
Oct 14, 2021, 3:42:25 PM10/14/21
to vim/vim, Subscribed

I'll include it now, thanks.
It would be nice to make an update of the whole file.

Bram Moolenaar

unread,
Oct 14, 2021, 3:42:28 PM10/14/21
to vim/vim, Subscribed

Closed #8900.

Reply all
Reply to author
Forward
0 new messages