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 ruby version in Netlify #1574

Merged
merged 2 commits into from
Aug 22, 2024

Conversation

bjohansebas
Copy link
Member

@bjohansebas bjohansebas commented Aug 16, 2024

After several attempts to fix this error and considering that it could be a version issue, as tried in PR (#1573) but in Docker, I noticed that Netlify uses an outdated default version. Although this change can be managed through environment variables, I don’t think it's best to handle it that way. Instead, it should be specified in a file so that administrators don’t have to manually change the version through the settings.

Resources:
https://answers.netlify.com/t/cant-deploy-with-ruby-3-1-2/55728
https://docs.netlify.com/configure-builds/available-software-at-build-time/
https://docs.netlify.com/configure-builds/manage-dependencies/#ruby

Close:
#1566

Edit: I just realized that the Docker configuration also needs to be fixed.
Edit: I'm not sure if these changes will work on GitHub Pages. If they don't work, it will be necessary to change the version using environment variables.
Edit: I have been testing in my repository, and GitHub Pages seems to work fine, so setting a specific Ruby version wouldn't be a problem.

Copy link

netlify bot commented Aug 16, 2024

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit 52b412c
🔍 Latest deploy log https://app.netlify.com/sites/expressjscom-preview/deploys/66bff82e1ccc5d00080ea195
😎 Deploy Preview https://deploy-preview-1574--expressjscom-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@bjohansebas bjohansebas changed the title try fix the netlify error fix ruby version in Netlify Aug 17, 2024
@bjohansebas bjohansebas marked this pull request as ready for review August 17, 2024 00:38
@bjohansebas bjohansebas marked this pull request as draft August 17, 2024 00:44
@bjohansebas bjohansebas marked this pull request as ready for review August 17, 2024 01:10
Copy link
Member

@UlisesGascon UlisesGascon left a comment

Choose a reason for hiding this comment

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

Thanks @bjohansebas for working on this!

@UlisesGascon
Copy link
Member

I will merge it as this will unblock previews and CI pipelines failing in many PRs

@UlisesGascon UlisesGascon merged commit e99a818 into expressjs:gh-pages Aug 22, 2024
6 checks passed
@bjohansebas bjohansebas deleted the fix-netlify-error branch August 31, 2024 16:35
chrisdel101 pushed a commit to chrisdel101/expressjs.com that referenced this pull request Nov 3, 2024
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