feat!: made EvaluationContext fields unexported with a constructor and setters to enforce immutability

[skyerus]

No description provided.

[codecov-commenter]

Codecov Report

Merging #91 (590dd7d) into main (9fbed88) will increase coverage by 0.43%.
The diff coverage is 83.78%.

@@            Coverage Diff             @@
##             main      #91      +/-   ##
==========================================
+ Coverage   69.24%   69.68%   +0.43%     
==========================================
  Files           7        8       +1     
  Lines         569      597      +28     
==========================================
+ Hits          394      416      +22     
- Misses        158      164       +6     
  Partials       17       17              

Impacted Files	Coverage Δ
pkg/openfeature/evaluation_context.go	80.95% <80.95%> (ø)
pkg/openfeature/client.go	69.41% <87.50%> (+0.03%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[james-milligan]

Nice work

[toddbaert]

This looks good.

I think the only thing I'm missing here is the lock on global providers and hooks, as well as client hooks. We probably want to read-lock all of these during evaluation, and write lock them during mutation.

[skyerus]

This looks good.

I think the only thing I'm missing here is the lock on global providers and hooks, as well as client hooks. We probably want to read-lock all of these during evaluation, and write lock them during mutation.

Great points. I've tackled this in another PR: #93