Skip to content

Commit 926466f

Browse files
committed
rabbit_feature_flags: Improve feature_props_extended() spec
The `provided_by` field is required. This permitted to discover a wrong spec of `rabbit_ff_registry:get/1`. While here, sort `feature_props_extended()` fields like `feature_props()`.
1 parent e207389 commit 926466f

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

deps/rabbit/src/rabbit_feature_flags.erl

+2-2
Original file line numberDiff line numberDiff line change
@@ -175,10 +175,10 @@
175175
-type feature_props_extended() :: #{desc => string(),
176176
doc_url => string(),
177177
stability => stability(),
178+
depends_on => [feature_name()],
178179
callbacks =>
179180
#{callback_name() => callback_fun_name()},
180-
depends_on => [feature_name()],
181-
provided_by => atom()}.
181+
provided_by := atom()}.
182182
%% The feature flag properties, once expanded by this module when feature
183183
%% flags are discovered.
184184
%%

deps/rabbit/src/rabbit_ff_registry.erl

+5-2
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@
6363
end).
6464

6565
-spec get(rabbit_feature_flags:feature_name()) ->
66-
rabbit_feature_flags:feature_props() | undefined.
66+
rabbit_feature_flags:feature_props_extended() | undefined.
6767
%% @doc
6868
%% Returns the properties of a feature flag.
6969
%%
@@ -75,7 +75,10 @@
7575

7676
get(FeatureName) ->
7777
_ = rabbit_ff_registry_factory:initialize_registry(),
78-
?convince_dialyzer(?MODULE:get(FeatureName), undefined, #{}).
78+
?convince_dialyzer(
79+
?MODULE:get(FeatureName),
80+
undefined,
81+
#{provided_by => rabbit}).
7982

8083
-spec list(all | enabled | disabled) -> rabbit_feature_flags:feature_flags().
8184
%% @doc

0 commit comments

Comments
 (0)