Refactor handling in HacsRepository to use the new API client#4902

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

fromuse-new

Open

Refactor handling in HacsRepository to use the new API client#4902

ludeeus wants to merge1 commit intomainfromuse-new

Conversation

Copy link

Member

ludeeus commentedNov 2, 2025

No description provided.

Refactor exception handling in HacsRepository to use the new API client

91bf63b

CopilotAI review requested due to automatic review settings

November 2, 2025 19:45

ludeeus added the pr: refactor label

Nov 2, 2025

ludeeus changed the title~~Refactor exception handling in HacsRepository to use the new API client~~Refactor handling in HacsRepository to use the new API client

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 updates the GitHub API integration by replacing the legacyAIOGitHubAPINotModifiedException exception with the newerGitHubNotModifiedException, and modernizes theasync_get_legacy_repository_object method to use the newgithubapi interface instead of the deprecatedget_repo method.

Key changes:

Updated exception handling to useGitHubNotModifiedException instead ofAIOGitHubAPINotModifiedException
Refactoredasync_get_legacy_repository_object to use the newself.hacs.githubapi.repos.get() API
Simplified exception handling by removing redundant exception types from the catch block

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