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

chore: update gems #381

Closed
wants to merge 1 commit into from
Closed

chore: update gems #381

wants to merge 1 commit into from

Conversation

RichardLitt
Copy link
Member

Just updating them all. Not sure why this didn't change the Gemfile, but whatever?

@RichardLitt
Copy link
Member Author

@XhmikosR Does this look right to you?

@RichardLitt RichardLitt requested a review from XhmikosR July 17, 2024 17:10
@XhmikosR
Copy link
Collaborator

XhmikosR commented Jul 17, 2024 via email

@RichardLitt
Copy link
Member Author

I think it is - still using old Node and older Ruby versions, and I'm not sure why. I would appreciate help!

@XhmikosR
Copy link
Collaborator

XhmikosR commented Jul 17, 2024 via email

@RichardLitt
Copy link
Member Author

Thank you! No rush at all. I appreciate your work.

@XhmikosR XhmikosR mentioned this pull request Jul 17, 2024
@XhmikosR XhmikosR deleted the feat/update-gems branch July 18, 2024 05:24
@XhmikosR
Copy link
Collaborator

@RichardLitt everything should be OK now. That .ruby-version file takes precedence over the Netlify dashboard environment variables so I had to update that too.

I only kept html-proofer back and eslint because they need more work and I don't have time. Generally, if CI is not green, the patch shouldn't land :)

@RichardLitt
Copy link
Member Author

Amazing! Thank you so much. :) This is great.

@RichardLitt
Copy link
Member Author

I wish I could look over your shoulder and see how you did this. There're some steps in here I don't fully understand. But again - grateful.

@XhmikosR
Copy link
Collaborator

I kept the patches separate for this reason :)

First CI, then gems, then npm deps and adapted the code if needed so that CI is still green.

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