Skip to content

Conversation

@johnlokerse
Copy link
Contributor

@johnlokerse johnlokerse commented Oct 16, 2020

Description

Fixed the variable check for Set-GitHubContent. This issue was mentioned by @amis92 and issued in #285

Issues Fixed

References

Checklist

@HowardWolosky HowardWolosky added api-contents Work to complete the API's defined here: https://developer.github.com/v3/repos/contents bug This relates to a bug in the existing module. labels Oct 16, 2020
Copy link
Contributor

@HowardWolosky HowardWolosky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Welcome to the project, @johnlokerse, and thanks for the fix!

@HowardWolosky
Copy link
Contributor

/azp run PowerShellForGitHub-CI

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@HowardWolosky HowardWolosky merged commit ec7950c into microsoft:master Oct 16, 2020
@HowardWolosky
Copy link
Contributor

/azp run PowerShellForGitHub-CI

@johnlokerse johnlokerse deleted the fix-issue-285 branch October 24, 2020 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api-contents Work to complete the API's defined here: https://developer.github.com/v3/repos/contents bug This relates to a bug in the existing module.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Copy-paste bug: Set-GitHubContent parameter checks on invalid variables

2 participants