Add new property to PullRequest ("merged" for forgejo compability) by HadronCollider · Pull Request #162 · moevm/github_repo_commitment_calc

HadronCollider · 2026-03-18T11:04:29Z

Наблюдение за смерженными PR в Forgejo:

поле "merged_by" - всегда пустое
поля "merged" (флаг) и "merged_at" (время мержа) используются и не пустые

Итого: в классы PR введено поле "merged" для идентификации смерженых PR (поле добавлено в конец csv-выгрузки, чтобы не ломать имеющиеся таблицы)

close #161

thehighestmath · 2026-03-27T12:45:53Z

/ai-review

thehighestmath · 2026-03-27T12:46:27Z

/ai-review

thehighestmath · 2026-03-27T12:47:46Z

/ai-review

thehighestmath · 2026-03-27T12:50:16Z

/ai-review

HadronCollider · 2026-04-15T15:17:52Z

Upd: информация о смержившем пользователе добавляется для смерженных PR (требуется дополнительный запрос для получения информации по конкретному PR)

HadronCollider · 2026-04-15T16:37:04Z

-                issue_url=None,  # TODO если возможно
+                merged_by=self.get_user_data(self.get_pull_request(repo, p.number).merged_by) if p.merged else None,
+                # TODO: merged_by always empty in result of repo_list_pull_requests
+                # (but merged and merged_at are usable)


Добавлена заглушка TODO для улучшения кода в будущем.

#ai-review-inline

HadronCollider · 2026-04-15T16:37:04Z

-                merged_by=self.get_user_data(p.merged_by) if p.merged_by else None,
-                files=[],  # TODO если возможно
-                issue_url=None,  # TODO если возможно
+                merged_by=self.get_user_data(self.get_pull_request(repo, p.number).merged_by) if p.merged else None,


Удалены строки, содержащие устаревшие поля merged_by, files и issue_url.

#ai-review-inline

HadronCollider · 2026-04-15T16:37:04Z

                repo.owner.login, repo.name, obj._id, 100000
-            )  # нет id комментария
+            )
+            # нет id комментария - сейчас не работает, т.к. требуется ID ревью - нужно сначала получить \


Добавлен комментарий о том, что сейчас не работает из-за отсутствия ID ревью.

#ai-review-inline

HadronCollider · 2026-04-15T16:37:04Z

-                merged_by=self.get_user_data(p.merged_by) if p.merged_by else None,
-                files=[],  # TODO если возможно
-                issue_url=None,  # TODO если возможно
+                merged_by=self.get_user_data(self.get_pull_request(repo, p.number).merged_by) if p.merged else None,


Добавлены строки с новыми полями merged_by и issue_url.

#ai-review-inline

HadronCollider · 2026-04-15T16:37:04Z

            comments = self.client.repository.repo_get_pull_review_comments(
                repo.owner.login, repo.name, obj._id, 100000
-            )  # нет id комментария
+            )


Изменено местоположение комментария, теперь он находится после запроса get_pull_review_comments.

#ai-review-inline

HadronCollider · 2026-04-15T16:37:04Z

            for p in pulls
        ]

+    @log_exceptions(default_return={}, message="Failed to get pull request data from Forgejo")


Добавлен метод get_pull_request для получения данных о Pull Request.

#ai-review-inline

HadronCollider · 2026-04-15T16:37:04Z

-                files=[],  # TODO если возможно
-                issue_url=None,  # TODO если возможно
+                merged_by=self.get_user_data(self.get_pull_request(repo, p.number).merged_by) if p.merged else None,
+                # TODO: merged_by always empty in result of repo_list_pull_requests


Добавлена заглушка TODO для улучшения кода в будущем.

#ai-review-inline

HadronCollider · 2026-04-15T16:37:05Z

-            )  # нет id комментария
+            )
+            # нет id комментария - сейчас не работает, т.к. требуется ID ревью - нужно сначала получить \
+            # список ревью /repos/{owner}/{repo}/pulls/{index}/reviews


Добавлен комментарий о необходимости отдельных запросов для получения комментариев и ревью.

#ai-review-inline

HadronCollider · 2026-04-15T16:37:05Z

+            )
+            # нет id комментария - сейчас не работает, т.к. требуется ID ревью - нужно сначала получить \
+            # список ревью /repos/{owner}/{repo}/pulls/{index}/reviews
+            # TODO: нужны отдельные запросы для получения комментариев и комментариев ревью #163, \


Упомянуты поля comments и review_comments в сущности PullRequest.

#ai-review-inline

HadronCollider · 2026-04-15T16:37:16Z

@@ -63,7 +63,7 @@ jobs:
            HEADER="repository name,author name,author login,author email,date and time,changed files,commit id,branch"
            ;;
          --pull_requests)


В коде была удалена строка с заголовком комментариев, но в новой версии файла она не обновлена.

#ai-review-inline

HadronCollider · 2026-04-15T16:37:16Z

            ;;
          --pull_requests)
-            HEADER="repository name,title,id,state,commit into,commit from,created at,creator name,creator login,creator email,changed files,comment body,comment created at,comment author name,comment author login,comment author email,merger name,merger login,merger email,source branch,target branch,assignee story,related issues,labels,milestone"
+            HEADER="repository name,title,id,state,commit into,commit from,created at,creator name,creator login,creator email,changed files,comment body,comment created at,comment author name,comment author login,comment author email,merger name,merger login,merger email,source branch,target branch,assignee story,related issues,labels,milestone,merged"


В коде добавлен новый заголовок для комментариев, но в новой версии файла нет указания на это.

#ai-review-inline

HadronCollider · 2026-04-15T16:37:24Z

            ;;
          --pull_requests)
-            HEADER="repository name,title,id,state,commit into,commit from,created at,creator name,creator login,creator email,changed files,comment body,comment created at,comment author name,comment author login,comment author email,merger name,merger login,merger email,source branch,target branch,assignee story,related issues,labels,milestone"
+            HEADER="repository name,title,id,state,commit into,commit from,created at,creator name,creator login,creator email,changed files,comment body,comment created at,comment author name,comment author login,comment author email,merger name,merger login,merger email,source branch,target branch,assignee story,related issues,labels,milestone,merged"


В коде была удалена строка с заголовком комментария, но в новой версии файла она не обновлена.

#ai-review-inline

HadronCollider · 2026-04-15T16:37:30Z

                    if pr["mergedBy"] and "email" in pr["mergedBy"]
                    else None
                ),
+                merged=pr["mergedBy"] and "name" in pr["mergedBy"],  # TODO: refactor?


Логика условного выражения в 'merged' не соответствует ожиданиям.

#ai-review-inline

HadronCollider · 2026-04-15T16:37:36Z

                head_ref=p.head.ref,
                base_ref=p.base.ref,
                merged_by=self.get_user_data(p.merged_by) if p.merged_by else None,
+                merged=bool(p.merged_by),


Добавлен логический флаг 'merged', который должен быть установлен в значение True, если есть автор слияния.

#ai-review-inline

HadronCollider · 2026-04-15T16:37:41Z

            merger_name=pull.merged_by.username if pull.merged_by else None,
            merger_login=pull.merged_by.login if pull.merged_by else None,
            merger_email=pull.merged_by.email if pull.merged_by else None,
+            merged=pull.merged,


Добавлен лишний комментарий в коде.

#ai-review-inline

HadronCollider · 2026-04-15T16:37:45Z

    related_issues: str = ''
    labels: str = ''
    milestone: str = ''
+    merged: bool = False


Добавлен логический флаг 'merged', который не имеет смысла в контексте данного класса.

#ai-review-inline

HadronCollider added 2 commits March 18, 2026 13:59

add new property to PullRequest ("merged" for forgejo compability)

e3051a8

add new PR col to smoke-tests

245abb0

HadronCollider self-assigned this Mar 18, 2026

update ForgejoRepoAPI.get_pull_requests to get merged_by info

c4bc89a

HadronCollider commented Apr 15, 2026

View reviewed changes

thehighestmath approved these changes Apr 16, 2026

View reviewed changes

Conversation

HadronCollider commented Mar 18, 2026

Uh oh!

thehighestmath commented Mar 27, 2026

Uh oh!

thehighestmath commented Mar 27, 2026

Uh oh!

thehighestmath commented Mar 27, 2026

Uh oh!

thehighestmath commented Mar 27, 2026

Uh oh!

HadronCollider commented Apr 15, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants