Skip to content

Environments and Variables #320

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 8 commits into from
Mar 16, 2025
Merged

Environments and Variables #320

merged 8 commits into from
Mar 16, 2025

Conversation

@pmatthews05
Copy link
Contributor

@pmatthews05 pmatthews05 commented Mar 13, 2025 •
edited by MariusStorhaug
Loading

Description

This includes new commands:

  • Get-GitHubVariable.ps1
  • Set-GitHubVariable.ps1
  • Remove-GitHubVariable.ps1
  • Get-GitHubEnvironment.ps1
  • Set-GitHubEnvironment.ps1
  • Remove-GitHubEnvironment.ps1

Type of change

  • 📖 [Docs]
  • 🪲 [Fix]
  • 🩹 [Patch]
  • ⚠️ [Security fix]
  • 🚀 [Feature]
  • 🌟 [Breaking change]

Checklist

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

Copilot AI review requested due to automatic review settings March 13, 2025 16:02
@pmatthews05 pmatthews05 requested a review from a team as a code owner March 13, 2025 16:02
Copilot AI reviewed Mar 13, 2025
View reviewed changes
Copy link
Contributor

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.

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

@pmatthews05
Copy link
Contributor Author

@MariusStorhaug I haven't been able to test this on my own machine. Unfortunately I cannot get the module running directly from the code. When I try to connect Connect-GitHubAccount using ClientID and PrivateKey, it is unable to Initialize-GitHubConfig giving the error message "Exception: An ID is required, either as a parameter or as a property of the context object"

!!!!! DO NOT MERGE THIS WITH YOUR MAIN CODE. THIS NEEDS CHECKING AND TESTING. !!!!!

@MariusStorhaug MariusStorhaug changed the base branch from main to mathews/env March 16, 2025 07:41
@MariusStorhaug MariusStorhaug requested a review from Copilot March 16, 2025 07:41
Copilot AI reviewed Mar 16, 2025
View reviewed changes
Copy link
Contributor

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.

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

@MariusStorhaug MariusStorhaug self-assigned this Mar 16, 2025
@MariusStorhaug MariusStorhaug moved this from Todo to In Progress in GitHub PowerShell Module Mar 16, 2025
@MariusStorhaug MariusStorhaug merged commit da7a926 into PSModule:mathews/env Mar 16, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in GitHub PowerShell Module Mar 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

@MariusStorhaug MariusStorhaug

Labels

None yet

Projects

GitHub PowerShell Module
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pmatthews05 @MariusStorhaug