ocabot migration duplicating lines in migration issue?

[sbidoul]

See for example OCA/server-env#120 where the issue was created correctly but now lines with references to a PR are duplicated.

[pedrobaeza]

I have observed it in some cases, but I'm not able to deduce what is reason. A problem in the regex?

[sbidoul]

Another example: OCA/partner-contact#1353

[eLBati]

Something similar is still happening:

OCA/l10n-italy#3026 (comment)
produced

[SirTakobi]

Something similar is still happening:

OCA/l10n-italy#3026 (comment) produced

That is because the alphabetical order of modules is checked in

oca-github-bot/src/oca_github_bot/tasks/migration_issue_bot.py

Line 60 in 44ad4a5

if line_module > module:

: when the bot reads l10n_it_ipa, it adds (before l10n_it_ipa) the l10n_it_declaration_of_intent line because 'l10n_it_ipa' > 'l10n_it_declaration_of_intent'.

TL;DR: the bot misbehaves when the modules are not written in alphabetical order, should we fix it?

[pedrobaeza]

It's not the same. This bot command assumes an alphabetical order in the migration issue, which is not the case.

[pedrobaeza]

You should order the modules IMO instead of over-complicating the bot.

[eLBati]

Thanks, ordered