Skip to content

Conversation

@wsobel
Copy link
Member

@wsobel wsobel commented Sep 28, 2025

Upgraded C++ to version 20 and boost to version 1.88.

@wsobel wsobel requested review from Copilot and simonyg September 28, 2025 20:20
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request upgrades the codebase from C++ version 17 to version 20 and updates Boost from version 1.85 to 1.88. The changes include updating compiler standards, modernizing deprecated Boost ASIO APIs, updating copyright years to 2025, and fixing compatibility issues with the new library versions.

Key Changes:

  • Updated C++ standard from 17 to 20 across all build configurations and profiles
  • Upgraded Boost dependency from 1.85.0 to 1.88.0 and other library versions
  • Replaced deprecated Boost ASIO APIs with modern equivalents throughout the codebase

Reviewed Changes

Copilot reviewed 291 out of 291 changed files in this pull request and generated no comments.

Show a summary per file
File Description
CMakeLists.txt Updated C++ standard to 20 and incremented build version
conanfile.py Upgraded Boost to 1.88.0, mqtt_cpp to 13.2.2, and gtest to 1.17.0
conan/profiles/* Updated all compiler profiles to use C++ standard 20
src/mtconnect/utilities.* Modernized date/time handling and fixed API compatibility issues
test_package/period_filter_test.cpp Replaced deprecated reset() with restart() for io_context
Various test and source files Updated copyright years to 2025 and fixed ASIO API usage
Comments suppressed due to low confidence (6)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@wsobel wsobel merged commit 8b21f2a into main Oct 3, 2025
7 checks passed
@wsobel wsobel deleted the migrate_to_cpp_version_20 branch October 3, 2025 19:11
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