Skip to content

Conversation

@wiso
Copy link

@wiso wiso commented Mar 3, 2025

No description provided.

@github-actions
Copy link

github-actions bot commented Mar 3, 2025

Errors and warnings found by git-po-helper in workflow #895:

------------------------------------------------------------------------------
ERROR [po/it.po@c33910b]	5013 translated messages. 
ERROR [po/it.po@c33910b]	too many obsolete entries (184) in comments, please remove them 
------------------------------------------------------------------------------
ERROR [po/it.po@c33910b]	Found file-location comments in po file. By submitting a location-less 
ERROR [po/it.po@c33910b]	"po/XX.po" file, the size of the Git repository can be greatly reduced. 
ERROR [po/it.po@c33910b]	See the discussion below: 
ERROR [po/it.po@c33910b]                           
ERROR [po/it.po@c33910b]	    https://lore.kernel.org/git/[email protected]/ 
ERROR [po/it.po@c33910b]                           
ERROR [po/it.po@c33910b]	As how to commit a location-less "po/XX.po" file, See: 
ERROR [po/it.po@c33910b]                           
ERROR [po/it.po@c33910b]	    the [Updating a "XX.po" file] section in "po/README.md" 
------------------------------------------------------------------------------
WARNING [po/it.po@c33910b]	mismatched patterns: estensioni.partialclone, extensions.partialclone 
WARNING [po/it.po@c33910b]	>> msgid: --filter can only be used with the remote configured in extensions.partialclone 
WARNING [po/it.po@c33910b]	>> msgstr: --filter può essere usato solo con il remoto configurato nelle estensioni.partialclone 
WARNING [po/it.po@c33910b]                           
WARNING [po/it.po@c33910b]	mismatched patterns: --mirror-fetch, --mirror-push, --mirror=fetch, --mirror=push 
WARNING [po/it.po@c33910b]	>> msgid: --mirror is dangerous and deprecated; please 
WARNING [po/it.po@c33910b]		 use --mirror=fetch or --mirror=push instead 
WARNING [po/it.po@c33910b]	>> msgstr: --mirror è pericoloso e deprecato; per favore 
WARNING [po/it.po@c33910b]		 usa invece --mirror-fetch o --mirror-push 
WARNING [po/it.po@c33910b]                           
WARNING [po/it.po@c33910b]	mismatched patterns: git rev-list, git-rev-list 
WARNING [po/it.po@c33910b]	>> msgid: Could not run 'git rev-list' 
WARNING [po/it.po@c33910b]	>> msgstr: Impossibile eseguire 'git-rev-list' 
WARNING [po/it.po@c33910b]                           
WARNING [po/it.po@c33910b]	mismatched patterns: refs/heads 
WARNING [po/it.po@c33910b]	>> msgid: HEAD not found below refs/heads! 
WARNING [po/it.po@c33910b]	>> msgstr: HEAD non trovato fra i riferimenti/head! 
WARNING [po/it.po@c33910b]                           
WARNING [po/it.po@c33910b]	mismatched patterns: index.Try 
WARNING [po/it.po@c33910b]	>> msgid: conflicts in index.Try without --index. 
WARNING [po/it.po@c33910b]	>> msgstr: ci sono conflitti nell'indice. Prova senza --index. 
WARNING [po/it.po@c33910b]                           
WARNING [po/it.po@c33910b]	mismatched patterns: --interactive, git rebase--interactive 
WARNING [po/it.po@c33910b]	>> msgid: git rebase--interactive [<options>] 
WARNING [po/it.po@c33910b]	>> msgstr: git rebase --interactive [<opzioni>] 
WARNING [po/it.po@c33910b]                           
WARNING [po/it.po@c33910b]	mismatched patterns: --verify-signature, --verify-signatures 
WARNING [po/it.po@c33910b]	>> msgid: ignoring --verify-signatures for rebase 
WARNING [po/it.po@c33910b]	>> msgstr: ignoro --verify-signature per il rebase 
WARNING [po/it.po@c33910b]                           
WARNING [po/it.po@c33910b]	mismatched patterns: --dirstat=files,param1,param2..., --dirstat=files,parametro1,parametro2... 
WARNING [po/it.po@c33910b]	>> msgid: synonym for --dirstat=files,param1,param2... 
WARNING [po/it.po@c33910b]	>> msgstr: sinonimo di --dirstat=files,parametro1,parametro2... 
WARNING [po/it.po@c33910b]                           
------------------------------------------------------------------------------
ERROR commit c33910b: subject ("fix sourgente") does not have prefix "l10n:" 
------------------------------------------------------------------------------
ERROR commit c33910b: empty body of the commit message, no s-o-b signature 
INFO checking commits: 0 passed, 1 failed.        
------------------------------------------------------------------------------
INFO downloading pot file from https://github.com/git-l10n/pot-changes/raw/pot/master/po/git.pot 
------------------------------------------------------------------------------
ERROR [po/it.po@c33910b]	1646 new string(s) in 'po/git.pot', but not in your 'po/XX.po' 
ERROR [po/it.po@c33910b]                           
ERROR [po/it.po@c33910b]	  > po/git.pot:545: this message is used but not defined in /tmp/3990681768--it.po 
ERROR [po/it.po@c33910b]	  > po/git.pot:597: this message is used but not defined in /tmp/3990681768--it.po 
ERROR [po/it.po@c33910b]	  > po/git.pot:602: this message is used but not defined in /tmp/3990681768--it.po 
ERROR [po/it.po@c33910b]	  > ...                   
ERROR [po/it.po@c33910b]                           
ERROR [po/it.po@c33910b]	827 obsolete string(s) in your 'po/XX.po', which must be removed 
ERROR [po/it.po@c33910b]                           
ERROR [po/it.po@c33910b]	  > po/XX.po:562: warning: this message is not used 
ERROR [po/it.po@c33910b]	  > po/XX.po:624: warning: this message is not used 
ERROR [po/it.po@c33910b]	  > po/XX.po:691: warning: this message is not used 
ERROR [po/it.po@c33910b]	  > ...                   
ERROR [po/it.po@c33910b]                           
ERROR [po/it.po@c33910b]	You can download the latest "po/git.pot" file from: 
ERROR [po/it.po@c33910b]                           
ERROR [po/it.po@c33910b]		https://github.com/git-l10n/pot-changes/raw/pot/master/po/git.pot 
ERROR [po/it.po@c33910b]                           
ERROR [po/it.po@c33910b]	Please rebase your branch to the latest upstream branch, 
ERROR [po/it.po@c33910b]	run "git-po-helper update po/XX.po" to update your po file, 
ERROR [po/it.po@c33910b]	and translate the new strings in it. 
ERROR [po/it.po@c33910b]                           

ERROR: fail to execute "git-po-helper check-commits"

@jiangxin
Copy link
Member

jiangxin commented Mar 3, 2025

It not necessary to close and create new PR, you can make a force-push to your branch, and the pull request will be refreshed.

@jiangxin
Copy link
Member

jiangxin commented Mar 3, 2025

[Reply for your message to the ML]

We have some conventions for pull requests to the git l10n repository,
and we use GitHub Actions to check pull requests according to these
conversions automatically. E.g.:

  1. The subject of your commit should start with 'l10n: '. You should
    amend your commit message; the title of the pull request does not
    matter, but the commit message is very important.

  2. Your patch is based on a old version of git.pot template file, and
    among the 5013 translated messages, there are about 1000 messages are
    obsolete. Fortunately, the message you modified still exist in the new
    version of the POT template file, and after you update the commit
    message title, I will merge your commit.

  3. There are obvious errors in old version of Italian translations,
    see the report of mismatched patterns in feedback of GitHub Actions:

    extensions.partialclone vs estensioni.partialclone
    git rev-list vs git-rev-list
    refs/heads vs riferimenti/head
    git rebase--interactive vs git rebase --interactive
    --verify-signatures vs --verify-signature

    But before changing the above errors, make sure you work on the new
    version of the POT template. You do not want to fix errors on obsolete
    messages.

  4. If you decide to contribute a larger patch based on the new POT
    template, please contribute a PO file without file locations, as
    frequently changing location information can significantly increase
    the size of the Git repository.

@jiangxin
Copy link
Member

jiangxin commented Mar 9, 2025

Hi @wiso

I created another PR #839 , which added prefix "l10n:" in the subject and add a S-O-B signature in the trailer.

Will close this PR and use #839 instead.

@jiangxin jiangxin closed this Mar 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants