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.

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

fix: handle 409 responses from GitHub Cache gracefully #24

Merged
merged 1 commit into from
May 1, 2024
Merged

fix: handle 409 responses from GitHub Cache gracefully #24

merged 1 commit into from
May 1, 2024

Conversation

Addono
Copy link
Contributor

@Addono Addono commented Mar 25, 2024

Handles HTTP 409 codes from GitHub Cache more gracefully, treating it as a cache-miss instead of failing the whole pipeline.

Resolves #23

@Addono Addono mentioned this pull request Mar 25, 2024
Closed
dtinth
dtinth approved these changes May 1, 2024
View reviewed changes
Copy link
Owner

@dtinth dtinth left a comment

Choose a reason for hiding this comment

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

This looks good to me, thank you very much! I appreciate your contribution.

Sorry it took so long for me to process this PR.

@dtinth dtinth merged commit cc723b4 into dtinth:main May 1, 2024
5 checks passed
@hirbod
Copy link

hirbod commented May 1, 2024

Awesome. Looking forward for the new version :)

@dtinth
Copy link
Owner

dtinth commented May 1, 2024

@hirbod v1.3.0 has been released.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dtinth dtinth dtinth approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Warn and handle failed cache-restoration (HTTP 500) instead of failing the pipeline
3 participants
@Addono @hirbod @dtinth