-
Notifications
You must be signed in to change notification settings - Fork 5
🚀 [Feature]: Add classes and restructure context handling #191
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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
…rted from the Context module
…Set-GitHubContext help documentation
… error management
…or improved debugging
…rules in PowerShell Script Analyzer configuration
…context properties
…nd clean up unused completer scripts
Contributor
|
Module [GitHub - 0.11.0] published to the PowerShell Gallery. |
Contributor
|
GitHub release for GitHub v0.11.0 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 introduces several new classes and refactors existing ones to enhance the GitHub context management in PowerShell scripts. Additionally, it includes changes to improve logging and configuration initialization. The most important changes are summarized below:
New Classes for GitHub Context Management:
src/classes/public/Config/GitHubConfig.ps1: Added a newGitHubConfigclass to manage GitHub configuration settings.src/classes/public/Context/GitHubContext.ps1: Added a newGitHubContextclass to encapsulate GitHub context details.GitHubContextan inherited class from theContextclass #193src/classes/public/Context/GitHubContext/AppGitHubContext.ps1: Added a newAppGitHubContextclass for managing GitHub App contexts.GitHubContext#190src/classes/public/Context/GitHubContext/InstallationGitHubContext.ps1: Added a newInstallationGitHubContextclass for managing installation contexts.GitHubContext#190src/classes/public/Context/GitHubContext/UserGitHubContext.ps1: Added a newUserGitHubContextclass for managing user contexts.GitHubContext#190Refactoring and Improvements:
src/classes/public/GitHubContext.ps1: Removed the oldGitHubContextclass, which has been replaced by the new context classes.src/functions/private/Auth/Context/Resolve-GitHubContext.ps1: Improved logging by changing the output format toFormat-Tableand added verbose logging for resolved contexts.src/functions/public/Auth/Connect-GitHubAccount.ps1: Refactored to use the newGitHubConfigand improved handling of default client IDs and tokens. [1] [2] [3] [4] [5]Configuration Initialization:
src/functions/private/Config/Initialize-GitHubConfig.ps1: Added a new functionInitialize-GitHubConfigto initialize GitHub configuration settings.Utility Enhancements:
src/functions/private/Utilities/PowerShell/Get-FunctionParameter.ps1: Enhanced theGet-FunctionParameterfunction to optionally return parameters as a hashtable. [1] [2] [3]Workflow Update:
.github/workflows/Linter.yml: Skipping validation for PowerShell scripts in the linter workflow.Type of change
Checklist