Skip to content

ライセンス取得関連対応#31

Merged
finphie merged 5 commits into
mainfrom
20250921
Sep 21, 2025
Merged

ライセンス取得関連対応#31
finphie merged 5 commits into
mainfrom
20250921

Conversation

@finphie
Copy link
Copy Markdown
Owner

@finphie finphie commented Sep 21, 2025

No description provided.

@finphie finphie requested a review from Copilot September 21, 2025 21:43
Copy link
Copy Markdown

Copilot AI 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 adds license-related functionality to the GitHub client library and performs some refactoring of documentation comments in partial classes.

  • Adds new ILicensesClient interface and implementation for retrieving repository license content
  • Refactors HTTP client configuration by extracting common functionality into separate methods
  • Updates XML documentation comments from <summary> to <content> tags for partial classes

Reviewed Changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
GitHubClientExtensions.cs Extracts common HTTP client configuration logic into separate methods
GitHubClient.cs Adds Licenses property and parameterless Create() method
GitHubClient.Licenses.cs Implements new license content retrieval functionality
ILicensesClient.cs Defines interface for license-related operations
IGitHubClient.cs Adds Licenses property to main interface
Multiple partial class files Updates documentation tags from <summary> to <content>
Solution files Migrates from .sln to .slnx format

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread Source/FToolkit.Net.GitHub/GitHubClient.Licenses.cs
Comment thread Source/FToolkit.Net.GitHub/GitHubClient.Licenses.cs
@finphie finphie merged commit 2a9a83c into main Sep 21, 2025
6 checks passed
@finphie finphie deleted the 20250921 branch September 21, 2025 21:48
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.

2 participants