[REF] create a dedicated function _mark_migration_done_in_migration_issue

Author: legalsylvain

src/oca_github_bot/tasks/merge_bot.py

@@ -27,7 +27,7 @@
 from ..utils import hide_secrets
 from ..version_branch import make_merge_bot_branch, parse_merge_bot_branch
 from .main_branch_bot import main_branch_bot_actions
-from .migration_issue_bot import _check_line_issue, _find_issue
+from .migration_issue_bot import _mark_migration_done_in_migration_issue
 
 _logger = getLogger(__name__)
 
@@ -204,10 +204,7 @@ def _merge_bot_merge_pr(org, repo, merge_bot_branch, cwd, dry_run=False):
         github.gh_call(gh_pr.close)
 
         # Check line in migration issue if required
-        migration_issue = _find_issue(gh_repo, target_branch)
-        if migration_issue:
-            new_body = _check_line_issue(gh_pr.number, migration_issue.body)
-            migration_issue.edit(body=new_body)
+        _mark_migration_done_in_migration_issue(gh_repo, target_branch, gh_pr)
     return True
 
 

src/oca_github_bot/tasks/migration_issue_bot.py

@@ -83,6 +83,13 @@ def _set_lines_issue(gh_pr_user_login, gh_pr_number, issue_body, module):
     return "\n".join(lines), old_pr_number
 
 
+def _mark_migration_done_in_migration_issue(gh_repo, target_branch, gh_pr):
+    migration_issue = _find_issue(gh_repo, target_branch)
+    if migration_issue:
+        new_body = _check_line_issue(gh_pr.number, migration_issue.body)
+        migration_issue.edit(body=new_body)
+
+
 @task()
 @switchable("migration_issue_bot")
 def migration_issue_start(org, repo, pr, username, module=None, dry_run=False):