Skip to content

Conversation

@pelesh
Copy link
Collaborator

@pelesh pelesh commented Jan 19, 2026

Description

Closes #331

CC @nkoukpaizan

Proposed changes

Include both, sstream and string in VariableMonitor.hpp.

Checklist

  • All tests pass.
  • Code compiles cleanly with flags -Wall -Wpedantic -Wconversion -Wextra.
  • The new code follows GridKit™ style guidelines.
  • There are unit tests for the new code.
  • N/A The new code is documented.
  • The feature branch is rebased with respect to the target branch.
  • N/A I have updated CHANGELOG.md to reflect the changes in this PR. If this is a minor PR that is part of a larger fix already included in the file, state so.

@pelesh pelesh requested a review from PhilipFackler January 19, 2026 19:17
@pelesh pelesh self-assigned this Jan 19, 2026
@pelesh pelesh added the bug Something isn't working label Jan 19, 2026
Copy link
Collaborator

@PhilipFackler PhilipFackler left a comment

Choose a reason for hiding this comment

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

Nice catch

@nkoukpaizan nkoukpaizan merged commit fed7fa3 into develop Jan 20, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build issue: Build with clang 16 fails with undefined template 'std::basic_ostringstream<char>' error

4 participants