Skip to content

Update deprecated windows build image#1483

Merged
sbmueller merged 2 commits intomasterfrom
smuller/fix-windoze
Jul 9, 2025
Merged

Update deprecated windows build image#1483
sbmueller merged 2 commits intomasterfrom
smuller/fix-windoze

Conversation

@sbmueller
Copy link
Copy Markdown
Contributor

@sbmueller sbmueller commented Jul 9, 2025

Description

@swift-nav/algint-team

API compatibility

Does this change introduce a API compatibility risk?
NO

API compatibility plan

If the above is "Yes", please detail the compatibility (or migration) plan:

JIRA Reference

https://swift-nav.atlassian.net/browse/BOARD-XXXX

@sbmueller sbmueller requested a review from a team as a code owner July 9, 2025 07:37
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Jul 9, 2025

@sbmueller sbmueller changed the title Fix windows build Update deprecated windows build image Jul 9, 2025
Copy link
Copy Markdown
Contributor

@martin4861 martin4861 left a comment

Choose a reason for hiding this comment

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

lgtm!

name: "Windows 2019 (Generator: ${{ matrix.generator }}, Shared Library: ${{ matrix.build_shared_libraries }})"
runs-on: windows-2019
name: "Windows Latest (Generator: ${{ matrix.generator }}, Shared Library: ${{ matrix.build_shared_libraries }})"
runs-on: windows-latest
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is there only windows-latest, or maybe something like windows-2025, which could give us some kind of reproducibility?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

There is both windows-2022 and windows-2025.
I thought: Specifying the version will 100% lead to build failures in the future, when the version gets deprecated again.
Using latest might give us a chance (maybe 10%?) that things keep on going smoothly without maintenance for quite a while.

@sbmueller sbmueller merged commit 7218b62 into master Jul 9, 2025
23 checks passed
@sbmueller sbmueller deleted the smuller/fix-windoze branch July 9, 2025 12:44
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.

3 participants