Skip to content

Conversation

@greghuels
Copy link
Collaborator

@greghuels greghuels commented Nov 21, 2025

See #23

@greghuels greghuels marked this pull request as ready for review November 21, 2025 17:22
@greghuels greghuels force-pushed the v2.0.0 branch 2 times, most recently from b1a270e to 7a22ece Compare November 21, 2025 17:42
@greghuels greghuels changed the title V2.0.0 v2.0.0 Nov 21, 2025
@greghuels greghuels force-pushed the v2.0.0 branch 2 times, most recently from bb4ab25 to dc790be Compare November 21, 2025 18:36
@greghuels greghuels changed the base branch from greg.huels/FFESUPPORT-332/no-exceptions to greg.huels/FFESUPPORT-334/exit-time-destructors November 21, 2025 18:37
@greghuels greghuels force-pushed the greg.huels/FFESUPPORT-334/exit-time-destructors branch from abea12e to 7a8bdc2 Compare November 21, 2025 19:41
Copy link
Contributor

@aarsilv aarsilv left a comment

Choose a reason for hiding this comment

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

📈

CHANGELOG.md Outdated
Comment on lines 12 to 16
- **BREAKING**: SDK now builds with `-fno-exceptions` and does not use exceptions internally
- **BREAKING**: All error handling is now done through logging and returning default values
- **BREAKING**: Removed `setIsGracefulFailureMode()` method - SDK always operates in "graceful" mode
- **BREAKING**: Removed custom exception classes (`FlagConfigurationNotFoundException`, `FlagNotEnabledException`, `SubjectAllocationException`)
- **BREAKING**: `EppoClient` constructor now takes `ConfigurationStore&` by reference instead of `std::shared_ptr<ConfigurationStore>`, allowing SDK consumers to choose their own synchronization strategy
Copy link
Contributor

Choose a reason for hiding this comment

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

For all those C++ SDK users 😛 (still good habit!)

@greghuels greghuels force-pushed the greg.huels/FFESUPPORT-334/exit-time-destructors branch from 7a8bdc2 to d592647 Compare November 25, 2025 12:50
@greghuels greghuels force-pushed the greg.huels/FFESUPPORT-334/exit-time-destructors branch 2 times, most recently from c55a49a to efa6d01 Compare November 25, 2025 13:27
@greghuels greghuels changed the base branch from greg.huels/FFESUPPORT-334/exit-time-destructors to main November 25, 2025 13:29
@greghuels greghuels force-pushed the v2.0.0 branch 2 times, most recently from 5d01b3b to 54dda5a Compare November 25, 2025 14:32
@greghuels greghuels force-pushed the v2.0.0 branch 3 times, most recently from 0a0a00c to e874bfc Compare November 25, 2025 18:04
@greghuels greghuels requested a review from dd-oleksii November 25, 2025 18:05
@greghuels greghuels merged commit 83be204 into main Nov 25, 2025
5 checks passed
@greghuels greghuels deleted the v2.0.0 branch November 25, 2025 18:34
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