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 busted preview-release build by forcing full ruby builds #458

Merged
merged 3 commits into from
Feb 7, 2024

Conversation

reese
Copy link
Collaborator

@reese reese commented Jan 9, 2024

This PR fixes the preview-release build by removing the ruby_checkout caching. This was causing issues with the ruby build, but removing it fixes any apparent discrepancies in the build that was causing issues. This probably makes it a bit slower, but I don't think speed is of the utmost priority since it's only run post-merge anyways 🤷

I verified this works by running it on this branch: result

@reese reese force-pushed the reese-preview-release branch 2 times, most recently from d5254a4 to 8860c5b Compare January 9, 2024 22:19
@reese reese changed the title [DNM] Fix busted preview-release build Fix busted preview-release build by removing ruby_checkout cache Jan 9, 2024
@reese reese requested a review from fables-tales January 9, 2024 22:22
@reese reese force-pushed the reese-preview-release branch from 87f5945 to 8860c5b Compare January 9, 2024 22:55
@reese reese force-pushed the reese-preview-release branch from c3a9f89 to 0f233f4 Compare January 22, 2024 17:16
@reese reese marked this pull request as draft January 23, 2024 15:45
@reese reese force-pushed the reese-preview-release branch from 0f233f4 to 6beb50a Compare January 26, 2024 16:15
@reese reese force-pushed the reese-preview-release branch from 6beb50a to 2e1d619 Compare January 26, 2024 16:17
@reese reese changed the title Fix busted preview-release build by removing ruby_checkout cache Fix busted preview-release build by forcing full ruby builds Jan 26, 2024
@reese
Copy link
Collaborator Author

reese commented Jan 26, 2024

I forced a test build for the release and made sure it passed: https://github.com/fables-tales/rubyfmt/actions/runs/7670725309

@reese reese marked this pull request as ready for review January 26, 2024 17:02
@reese reese merged commit 48f336e into trunk Feb 7, 2024
7 checks passed
@reese reese deleted the reese-preview-release branch February 7, 2024 16:58
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.

3 participants