search

open-feature / spec

OpenFeature specification
https://openfeature.dev
Apache License 2.0
597 stars 35 forks source link

Evaluation Context Merging 3.2.2 is missing before hooks #131

Closed beeme1mr closed 1 year ago

beeme1mr commented 1 year ago

Requirement 3.2.2 should included the before hook.

API (global) -> client -> invocation -> before hooks

An explanation of why the before hook has the highest priority should be included in the non-normative section.

toddbaert commented 1 year ago

I mentioned this in a previous PR...

I'm not sure I agree. This doesn't really have much to do with hooks - it's really about setting context at various levels. Personally, I would rather get rid of this requirement entirely then basically have the exact same information in 2 places.

toddbaert commented 1 year ago

Maybe we could just have a link in the non-normative section of 3.2.1 to 4.3.4?