Commit 00a4e4a
authored
fix: Remove virtual GetEventChannel from FeatureProvider (#401)
<!-- Please use this template for your pull request. -->
<!-- Please use the sections that you need and delete other sections -->
## This PR
<!-- add the description of the PR here -->
This pull request includes changes to the `EventChannel` handling in the
`OpenFeature` library to improve code consistency and reliability.
Improvements to `EventChannel` handling:
*
[`src/OpenFeature/EventExecutor.cs`](diffhunk://#diff-f563baadcf750bb66efaea76d7ec4783320e6efdc45c468effb531c948a2292cL150-R150):
Removed unnecessary null-conditional operator when completing the
`Writer` of the event channel.
*
[`src/OpenFeature/FeatureProvider.cs`](diffhunk://#diff-96ebc8fc507d0a19d55b9a5cb57b72a0e8058e09f31ee7d0b39e99b00c5029d8L143-R143):
Made the `GetEventChannel` method non-virtual to ensure consistent
behavior across different implementations.
### Related Issues
<!-- add here the GitHub issue that this PR resolves if applicable -->
Related to #358
---------
Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>1 parent 46274a2 commit 00a4e4a
2 files changed
+2
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
147 | 147 | | |
148 | 148 | | |
149 | 149 | | |
150 | | - | |
| 150 | + | |
151 | 151 | | |
152 | 152 | | |
153 | 153 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
140 | 140 | | |
141 | 141 | | |
142 | 142 | | |
143 | | - | |
| 143 | + | |
144 | 144 | | |
145 | 145 | | |
146 | 146 | | |
| |||
0 commit comments