Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Aug 13, 2025

Added missing exception documentation for the StoreContext.GetAppLicenseAsync method to address undocumented error cases.

Changes Made

Added an ## -exceptions section to document the ERROR_NOT_FOUND exception that can be thrown when the app license could not be found or retrieved. The documentation follows the established patterns used by other Store methods in the same namespace.

Specific addition:

  • Documents System.Runtime.InteropServices.COMException with error code 0x80070490 (ERROR_NOT_FOUND)
  • Provides clear explanation of when this exception occurs
  • Maintains consistency with existing documentation format

Context

The issue reported that error cases were not documented for this method, specifically mentioning that it "can at a minimum throw an ERROR_NOT_FOUND exception under some circumstances." This change addresses that gap by providing explicit documentation for developers using this API.

The documentation now matches the format used by similar Store methods like RequestPurchaseAsync and RequestRateAndReviewAppAsync, ensuring consistency across the namespace.

Fixes #995.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copy link
Contributor Author

Copilot AI commented Aug 13, 2025

@GrantMeStrength 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

@prmerger-automator prmerger-automator bot added the do-not-merge PR is being processed and awaiting review/approval. label Aug 13, 2025
Copilot AI changed the title [WIP] Error cases are not documented Document ERROR_NOT_FOUND exception for StoreContext.GetAppLicenseAsync Aug 13, 2025
Copilot AI requested a review from GrantMeStrength August 13, 2025 16:46
Copilot finished work on behalf of GrantMeStrength August 13, 2025 16:46
@GrantMeStrength GrantMeStrength removed the do-not-merge PR is being processed and awaiting review/approval. label Aug 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error cases are not documented

2 participants