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: update to use ubuntu-latest runners #11731

Merged
merged 1 commit into from
Oct 4, 2022
Merged

Conversation

vince-fugnitto
Copy link
Member

What it does

The pull-request updates CI to replace usages of ubuntu-18.04 which is deprecated and soon to be removed, and instead opts to use ubuntu-latest. The changes also align ubuntu-latest across the repo.

Additional Info:

How to test

Confirm that CI passes.

Review checklist

Reminder for reviewers

Signed-off-by: vince-fugnitto [email protected]

The commit updates CI to replace usages of `ubuntu-18.04` which is
deprecated and soon to be removed, and instead opts to use
`ubuntu-latest`. The changes also align `ubuntu-latest` across the repo.

Signed-off-by: vince-fugnitto <[email protected]>
@vince-fugnitto vince-fugnitto added the ci issues related to CI / tests label Sep 30, 2022
Copy link
Member

@paul-marechal paul-marechal left a comment

Choose a reason for hiding this comment

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

I was misunderstanding ubuntu-latest as ubuntu-most-most-recent-version but it seems to actually target the "oldest" LTS which is perfect for CI IMO. LGTM.

@vince-fugnitto vince-fugnitto merged commit afe5f3e into master Oct 4, 2022
@vince-fugnitto vince-fugnitto deleted the vf/ci-ubuntu branch October 4, 2022 16:33
@github-actions github-actions bot added this to the 1.31.0 milestone Oct 4, 2022
planger added a commit to planger/theia-e2e-test-suite that referenced this pull request Jan 2, 2023
planger added a commit to planger/theia-e2e-test-suite that referenced this pull request Jan 2, 2023
See related Theia changes
  * eclipse-theia/theia#11845
  * eclipse-theia/theia#11731

Contributed on behalf of STMicroelectronics.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci issues related to CI / tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants