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

ci: use phpstan result cache #3163

Merged

Conversation

wescopeland
Copy link
Member

Another PR to improve CI performance. This specifically targets analyse (PHPStan) while running in GitHub Actions.

As the project grows and grows, the analyse step is going to take exponentially longer because, currently, there is no cache to leverage in the actions containers.

This branch incorporates the official instructions for setting up the PHPStan cache in GitHub Actions from https://phpstan.org/user-guide/result-cache#setup-in-github-actions.

@wescopeland wescopeland requested a review from a team January 31, 2025 22:45
@wescopeland
Copy link
Member Author

wescopeland commented Jan 31, 2025

https://github.com/RetroAchievements/RAWeb/actions/runs/13082072305/job/36507524947?pr=3163 confirms a successful cache hit. The cache will be available across all branches.

https://github.com/RetroAchievements/RAWeb/actions/runs/13082104609/job/36507621934?pr=3163 is an example of a super fast run from a cache hit.

@wescopeland wescopeland merged commit e53035b into RetroAchievements:master Feb 1, 2025
9 checks passed
@wescopeland wescopeland deleted the ci-analyse-result-cache branch February 1, 2025 15:30
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.

2 participants