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

Fix checking state cache (fix #1136), also switch to octokit methods #1152

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

itchyny
Copy link

@itchyny itchyny commented Apr 10, 2024

Description:
This PR fixes #1136, also fixes a TODO comment about using octokit methods.

Related issue:

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.
    • These methods are not covered by existing tests, possibly because they interact with the GitHub APIs.

@itchyny itchyny requested a review from a team as a code owner April 10, 2024 08:54
@Georgegriff
Copy link

I think i ran into this issue, do you know if its likely this pr might progress?

@itchyny
Copy link
Author

itchyny commented Jun 24, 2024

I hope some maintainer will look into this PR, but maintenance activity of this action looks to be stale.

@echang49
Copy link

echang49 commented Aug 2, 2024

Would love for a maintainer to merge this fix in. Have a feature that's built on top of this that's needed internally and will definitely help others.

@echang49 echang49 mentioned this pull request Aug 2, 2024
2 tasks
@Dreamsorcerer
Copy link

Dreamsorcerer commented Oct 1, 2024

@Jcambass @HarithaVattikuti @priyagupta108 Can someone merge this PR? The action is completely broken for many repos without this. We have stale issues that should have been closed weeks ago, but will never close until this fix is released.

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.

State restoration fails if a repo has many caches
4 participants