Skip to content

Conversation

@Wave0004
Copy link
Contributor

No description provided.

@cyberc001
Copy link
Contributor

Это относится к #103? Потому что в issue написано, что нужно оптимизировать метод get_comments().
И не понял логики кэширования.

self._issue_cache[repo._id] = issue_dict

Здесь задаётся поле _issue_cache, но оно нигде не используется. Нужно проверять _issue_cache is None, и если оно None, то записывать в кэш, иначе просто считывать issues оттуда (кроме того, get_issues() не надо оптимизировать, надо оптимизировать get_comments()).

@Wave0004
Copy link
Contributor Author

Неверно понял, переделал

@thehighestmath
Copy link
Collaborator

12 часов назад сделан ПР, а уже конфликты есть
Последный кормит в мастер был 2 недели назад

@cyberc001 cyberc001 self-requested a review April 17, 2025 12:18
@cyberc001 cyberc001 merged commit 2e7310e into master Apr 17, 2025
9 checks passed
@thehighestmath thehighestmath linked an issue May 11, 2025 that may be closed by this pull request
@thehighestmath thehighestmath deleted the 103-optimized-pull_request-and-issue branch May 11, 2025 14:13
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.

Оптимизировать GithubRepoAPI

4 participants