Skip to content

Conversation

@sbmueller
Copy link
Contributor

@sbmueller sbmueller commented Oct 15, 2025

Description

@swift-nav/algint-team

This updates the few C++ targets in this repo to C++17. The change consists of

  • Update Bazel and CMake build systems to compile C++ targets according to the C++17 standard

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/AP-3454

@sbmueller sbmueller marked this pull request as ready for review October 15, 2025 16:21
@sbmueller sbmueller requested a review from a team as a code owner October 15, 2025 16:21
@sbmueller sbmueller force-pushed the smuller/AP-3454-c++17 branch 2 times, most recently from a67f626 to 6a35a77 Compare October 16, 2025 07:12
@sbmueller sbmueller force-pushed the smuller/AP-3454-c++17 branch from 6a35a77 to 2521cd5 Compare October 16, 2025 07:14
@sonarqubecloud
Copy link

Copy link
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!

@sbmueller sbmueller merged commit 253c925 into master Oct 16, 2025
23 checks passed
@sbmueller sbmueller deleted the smuller/AP-3454-c++17 branch October 16, 2025 07:47
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.

4 participants