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

Upgrade to Debian Stable and Ruby 3.2 #943

Merged
merged 4 commits into from
Jun 11, 2024
Merged

Conversation

benjamineskola
Copy link
Contributor

@benjamineskola benjamineskola commented Jun 6, 2024

No description provided.

@pixeltrix
Copy link
Contributor

Probably want to upgrade the GHA config to use Ruby 3.2 and ubuntu-24.04

@benjamineskola benjamineskola force-pushed the ruby-3.2 branch 6 times, most recently from 4f97a54 to 912da6f Compare June 6, 2024 10:58
@pixeltrix
Copy link
Contributor

All the errors seem related to the Active Storage image on the debate outcome models

@benjamineskola
Copy link
Contributor Author

All the errors seem related to the Active Storage image on the debate outcome models

I can't reproduce this locally (in docker) — wondering if there's some difference between Debian and Ubuntu?

Copy link
Contributor

@pixeltrix pixeltrix left a comment

Choose a reason for hiding this comment

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

We probably want to upgrade bundler as well, but that'll need changing in the server config as well

- name: Install image processing tools
shell: bash
run: |
sudo apt-get -yqq install imagemagick libvips
Copy link
Contributor

Choose a reason for hiding this comment

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

Interesting this is not in the default setup - seems to be a change in the 24.04 runner vs the 22.04 runner

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I'm not sure what the underlying change is — perhaps it was previously pulled in as a dependency by something else? Can't hurt to be explicit about it everywhere though.

@pixeltrix pixeltrix merged commit f96b5ea into alphagov:master Jun 11, 2024
12 checks passed
@pixeltrix pixeltrix deleted the ruby-3.2 branch June 11, 2024 09:40
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