Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: fix comment in README for Hook’s after method #1102

Merged
merged 2 commits into from
Dec 10, 2024

Conversation

chrisllontop
Copy link
Contributor

This PR

  • Fixes incorrect comment in the README for the Hook implementation's after method.

Signed-off-by: Christian Llontop <me@chrisllontop.com>
@chrisllontop chrisllontop requested a review from a team as a code owner December 10, 2024 19:00
@chrisllontop chrisllontop changed the title docs: fix incorrect comment in README for Hook’s after method docs: fix comment in README for Hook’s after method Dec 10, 2024
@toddbaert toddbaert merged commit ba8d1ae into open-feature:main Dec 10, 2024
2 of 4 checks passed
@chrisllontop chrisllontop deleted the patch-1 branch December 10, 2024 19:14
beeme1mr added a commit that referenced this pull request Dec 18, 2024
🤖 I have created a release *beep* *boop*
---


##
[1.4.0](web-sdk-v1.3.2...web-sdk-v1.4.0)
(2024-12-18)

### ⚠ BREAKING CHANGES

The signature of the `finally` hook stage has been changed. The
signature now includes the `evaluation details`, as per the [OpenFeature
specification](https://openfeature.dev/specification/sections/hooks#requirement-438).
Note that since hooks are still `experimental,` this does not constitute
a change requiring a new major version. To migrate, update any hook that
implements the `finally` stage to accept `evaluation details` as the
second argument.

* add evaluation details to finally hook
([#1087](#1087))
([2135254](2135254))

### ✨ New Features

* add evaluation details to finally hook
([#1087](#1087))
([2135254](2135254))


### 📚 Documentation

* fix comment in README for Hook’s after method
([#1102](#1102))
([ba8d1ae](ba8d1ae))


### 🔄 Refactoring

* improve track interface for providers
([#1100](#1100))
([5e5b160](5e5b160))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Signed-off-by: OpenFeature Bot <109696520+openfeaturebot@users.noreply.github.com>
Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
Co-authored-by: Michael Beemer <beeme1mr@users.noreply.github.com>
kevinmlong pushed a commit to kevinmlong/openfeature-js-sdk that referenced this pull request Dec 21, 2024
🤖 I have created a release *beep* *boop*
---


##
[1.4.0](open-feature/js-sdk@web-sdk-v1.3.2...web-sdk-v1.4.0)
(2024-12-18)

### ⚠ BREAKING CHANGES

The signature of the `finally` hook stage has been changed. The
signature now includes the `evaluation details`, as per the [OpenFeature
specification](https://openfeature.dev/specification/sections/hooks#requirement-438).
Note that since hooks are still `experimental,` this does not constitute
a change requiring a new major version. To migrate, update any hook that
implements the `finally` stage to accept `evaluation details` as the
second argument.

* add evaluation details to finally hook
([open-feature#1087](open-feature#1087))
([2135254](open-feature@2135254))

### ✨ New Features

* add evaluation details to finally hook
([open-feature#1087](open-feature#1087))
([2135254](open-feature@2135254))


### 📚 Documentation

* fix comment in README for Hook’s after method
([open-feature#1102](open-feature#1102))
([ba8d1ae](open-feature@ba8d1ae))


### 🔄 Refactoring

* improve track interface for providers
([open-feature#1100](open-feature#1100))
([5e5b160](open-feature@5e5b160))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Signed-off-by: OpenFeature Bot <109696520+openfeaturebot@users.noreply.github.com>
Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
Co-authored-by: Michael Beemer <beeme1mr@users.noreply.github.com>
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