Skip to content

Bump Ruby to 2.7.4, update github-pages and html-proofer gems #826

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

Merged
merged 2 commits into from
Apr 21, 2023

Conversation

Sjors
Copy link
Member

@Sjors Sjors commented Dec 17, 2021

This requires updating the server to Debian Stable Bullseye, cc TheBlueMatt.

I tested a few pages locally and didn't notice any difference.

I also ran the htmlproofer incantation from the README, which I updated to match Makefile. Only one minor change was needed to satisfy the updated version.

Closes #825

@laanwj
Copy link
Member

laanwj commented Feb 3, 2022

Concept ACK. It makes sense to use a supported version of Ruby for easier local reproducibility.

@RandyMcMillan

This comment was marked as outdated.

@katesalazar
Copy link
Contributor

I'm planning not to wipe my Buster hosts out until summer, sorry. On the bright side, summer's almost here! 🏖️

@fanquake fanquake mentioned this pull request Nov 25, 2022
@Sjors
Copy link
Member Author

Sjors commented Nov 28, 2022

I'll wait for that to be merged

fanquake added a commit that referenced this pull request Nov 28, 2022
04e9f6b README: add --check-html to htmlproofer (Sjors Provoost)
488ae09 Satisfy newer html-proofer (Sjors Provoost)

Pull request description:

  Cherry-picks two commits from #826 (which needs rebase, and is dependant on infra updates), which are correct, and can be merged now.

ACKs for top commit:
  stickies-v:
    ACK 04e9f6b

Tree-SHA512: 84048d6acd9126c75583e5d891a737e9ff65652ac8f30df160d66487d858ce5b917d401edbf73c0c882c4139bc7f63ff07c0790cc612fd5762c40603a4d9beba
Sjors added 2 commits December 1, 2022 16:51
Ruby 2.5 is EOL and Debian Stable ships 2.7.4
The github-pages and html-proofer gems we use are not specified by semver, so this just bumps them to their most recent versions:
* github-pages: 202 to 222
* html-proofer: 3.14.1 to 3.19.2
@Sjors
Copy link
Member Author

Sjors commented Dec 1, 2022

Rebased. I can bump the Ruby (patch) version further if we want to use a more modern version of Debian.

@fanquake fanquake mentioned this pull request Mar 27, 2023
Copy link
Contributor

@pinheadmz pinheadmz left a comment

Choose a reason for hiding this comment

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

ACK bda840a

Tested locally on macOS. This also should fix CI failures on important updates like #957

Show Signature
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

ACK bda840a988310629ab677370c80ec93eb3df6ec5
-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEE5hdzzW4BBA4vG9eM5+KYS2KJyToFAmQhx1MACgkQ5+KYS2KJ
yTqzgBAAiqilomgkqdJ02zQeySa/avQVQFl0MRMyutvtW/3Hayvdhg5YNKBqlyvn
gAgVjhMP/RKA2nfJPdeWiXzb9JiRoURJHqqtRbWXL5Gi/F9idfPkEqp28qSklPX4
ZgMZU5hWIHsqjy5Q9t7p4cer6qDxAJaUk2+0TwaXfVvkSTHQzVJr9/juefh21wJR
jDXHaynqjb+jUF3C4gyAbJmIlptffdQJ/uE3iZJJHNixJP+W6sN55RbIogwNO8lA
HaKhhYOaYXUtKDweKFl0S5WmTwltG3w6j/3nTXrYS9OqglGXfU3l9l5PQ/+Go/09
0ie+d4XBMrUyGN/1vDEyW1cXqK7V5CSbYyG6CtjvJbkQZrBSLHEX/jMzMkfa35j7
KpxP2FrLISG9lIlqram+LauhC3SMMm1JcRRkB5OcVflGQjbCLHgl54EBSnDp3+hO
ZGHgKVe9j1PhnWc7KsdH3jAgJLjHKb949VpZRl5vMgb5xxPl5NkAQEWucAeZYqGC
dNykxNofS28bflEjzRLpy38BceNiPjm3+2KXSD4ZA8m29phKVANSOQPNzhNC4mgU
QBsviGw+Q+MtM4BUbNmAPaeG5Fn87X+wB93hK9zS9MMkvmnFtaKryuAc6WOFY8+L
BkvCFK6WIq1SF8/Pg9MRXnuVL7CWyZEBev/gEQu2/l7uKpijs1w=
=Dy6W
-----END PGP SIGNATURE-----

pinheadmz's public key is on keybase

@fanquake
Copy link
Member

Going to merge this just in time for Ruby 2.7 to also go EOL: https://www.ruby-lang.org/en/news/2023/03/30/ruby-2-7-8-released/.

@fanquake fanquake merged commit 535107a into bitcoin-core:master Apr 21, 2023
@Sjors Sjors deleted the 2021/12/ruby branch April 24, 2023 08:08
@Sjors
Copy link
Member Author

Sjors commented Apr 24, 2023

I'll make a PR to bump to 2.7.8 so we can stay on the bleeding edge of the EOL.

Any gems that need bumping?

@fanquake
Copy link
Member

There's no need for a PR. As as far as I'm aware, that version of ruby won't be available.

@Sjors
Copy link
Member Author

Sjors commented Apr 24, 2023

Mmm, the release mentions it fixes security issues. Anyway, I can (also) make a PR to the 3.1 or 3.2 series. That involves a bit more testing though.

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.

Ruby 2.5 is EOL
6 participants