Skip to content
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

Update Get-ALZGithubRelease.ps1 #86

Closed

Conversation

MarkFarkas
Copy link

@MarkFarkas MarkFarkas commented Oct 16, 2023

Add support for pagination of Github releases

Pull Request

Issue

Fixes #85

Description

Description of changes: Add support for pagination when calling the releases endpoint on the GitHub API.

License

By submitting this pull request, I confirm that my contribution is made under the terms of the projects associated license.

Add support for pagination of Github releases
@oZakari
Copy link
Contributor

oZakari commented Oct 16, 2023

Hi @MarkFarkas, thanks for creating this PR. Could you update the unit tests for this change as it looks like the while loop is creating an infinite loop with the current test configuration.

See https://github.com/Azure/ALZ-PowerShell-Module/actions/runs/6536909020/job/17749625852?pr=86

@oZakari oZakari closed this Oct 16, 2023
@oZakari oZakari reopened this Oct 16, 2023
@oZakari
Copy link
Contributor

oZakari commented Oct 16, 2023

Never mind, looks like the requests are getting blocked by GitHub with a 403 forbidden response. Just going to wait a bit before running again as there may be some issues with GitHub atm as I am experiencing issues locally as well.

@oZakari oZakari closed this Oct 16, 2023
@oZakari oZakari reopened this Oct 16, 2023
@oZakari
Copy link
Contributor

oZakari commented Oct 17, 2023

Function refactored with the following PR: #87

@oZakari oZakari closed this Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Get-ALZGithubRelease.ps1 doesn't retrieve all releases from Github
2 participants