🪲 [Fix]: GitHub App Installations load failure #200
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
|
Module [GitHub - 0.11.6] published to the PowerShell Gallery. |
Contributor
|
GitHub release for GitHub v0.11.6-apps31 has been created. |
Contributor
|
Module [GitHub - 0.11.6] published to the PowerShell Gallery. |
Contributor
|
GitHub release for GitHub v0.11.6-apps32 has been created. |
…ntext name structure
Contributor
|
Module [GitHub - 0.11.6] published to the PowerShell Gallery. |
Contributor
|
GitHub release for GitHub v0.11.6-apps33 has been created. |
Contributor
|
Module [GitHub - 0.11.6] published to the PowerShell Gallery. |
Contributor
|
GitHub release for GitHub v0.11.6-apps34 has been created. |
Contributor
|
Module [GitHub - 0.11.6] published to the PowerShell Gallery. |
Contributor
|
GitHub release for GitHub v0.11.6-apps35 has been created. |
…Events, and Token
…based on existing values
…n and New-GitHubAppInstallationAccessToken calls
…g of context properties
…prove verbose logging for better clarity
… in Set-GitHubContext
…bosity for clarity
MariusStorhaug
temporarily deployed
to
github-pages
GitHub Actions
Inactive
Contributor
|
Module [GitHub - 0.11.6] published to the PowerShell Gallery. |
Contributor
|
GitHub release for GitHub v0.11.6 has been created. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This pull request includes several changes to improve the handling of GitHub contexts and enhance the functionality of the
Connect-GitHubAccountandConnect-GitHubAppfunctions. The most important changes include updating the type of thePermissionsproperty, adding parameterless constructors, and modifying the logic for loading GitHub App contexts.Enhancements to GitHub context handling:
src/classes/public/Context/GitHubContext/AppGitHubContext.ps1: Changed the type of thePermissionsproperty fromstring[]topscustomobjectand added a parameterless constructor.src/classes/public/Context/GitHubContext/InstallationGitHubContext.ps1: Changed the type of thePermissionsproperty fromstring[]topscustomobjectand added a parameterless constructor. [1] [2]src/classes/public/Context/GitHubContext/UserGitHubContext.ps1: Added a parameterless constructor.Improvements to
Connect-GitHubAccountfunction:src/functions/public/Auth/Connect-GitHubAccount.ps1: Renamed parameters (SkipAppAutoloadtoAutoloadInstallations), removed unused parameters (Shallow), and updated the logic for loading GitHub App contexts. [1] [2] [3] [4]Enhancements to
Connect-GitHubAppfunction:src/functions/public/Auth/Connect-GitHubApp.ps1: Removed theShallowparameter, added context resolution and validation in theprocessblock, and updated the logic for processing installations. [1] [2]Refinements to
Set-GitHubContextfunction:src/functions/public/Auth/Context/Set-GitHubContext.ps1: Refactored to use a local variable for context manipulation and added detailed logging for context information.Coverage documentation updates:
Coverage.md: Updated the number of available and missing functions, and adjusted the coverage percentage. [1] [2]Type of change
Checklist