Refactor last commit retrieval to use a non deprecated method#4898

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Open

ludeeus wants to merge1 commit intomain

base:main

Choose a base branch

fromchange-retrival-of-commit

Open

Refactor last commit retrieval to use a non deprecated method#4898

ludeeus wants to merge1 commit intomainfromchange-retrival-of-commit

Conversation

Copy link

Member

ludeeus commentedNov 2, 2025

The entire repository_object is deprecated in aiogithubapi; this is a small step in getting rid of it.

Refactor last commit retrieval to use a non deprecated method

bd40f7d

CopilotAI review requested due to automatic review settings

November 2, 2025 13:29

ludeeus added the pr: refactor label

Nov 2, 2025

CopilotAI reviewed

Nov 2, 2025

View reviewed changes

Copy link

Contributor

CopilotAI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Pull Request Overview

This PR refactors the code to consolidate duplicate logic for fetching repository commit information. The main change replaces direct calls torepository_object.set_last_commit() with a new methodasync_set_last_commits() that uses the GitHub REST API to fetch branch information.

Introduces a new methodasync_set_last_commits() that fetches the last commit SHA from the repository's default branch
Replaces two instances of duplicated commit-fetching code with calls to the new method
Changes the implementation from usingrepository_object.set_last_commit() to directly calling the GitHub API

💡Add Copilot custom instructions for smarter, more guided reviews.Learn how to get started.