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

ci: Change Docker images to ubuntu:24.10 #4119

Closed
wants to merge 11 commits into from

Conversation

tgross35
Copy link
Contributor

23.10 is EOL so update to the latest stable version.

23.10 is EOL so update to the latest stable version.
@rustbot
Copy link
Collaborator

rustbot commented Nov 20, 2024

r? @JohnTitor

rustbot has assigned @JohnTitor.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

`ctest` sometimes reports a parsing failure but it isn't consistent (I
do not know why). Just use a less ideal older syntax for now, this is an
internal macro.
@tgross35
Copy link
Contributor Author

NOT FUN 24.04 is also when they did the change to 64-bit time_t which we are not ready for...

Since 6ab46bb ("ci: Use some tricks..."), `rustfmt` is invoked
manually on all files in `src`. However, this stopped running `cargo
fmt` which covered everything that wasn't in `src`.

This was unintentional so add it back.
@tgross35
Copy link
Contributor Author

Superseded by #4120

@tgross35 tgross35 closed this Nov 20, 2024
@tgross35 tgross35 deleted the latest-ubuntu branch November 20, 2024 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants