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

fix(typespec): fix hocon_schema:override typespec #289

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

thalesmg
Copy link
Contributor

We commonly use deprecated => {since, "x.y.z"}, but that is not what the typespec expects.

Line 72 Column 50: The call hocon_schema:override(Sc::atom() | fun((_) -> any()) | maybe_improper_list() | tuple() | map(), Override::#{'default':=[], 'deprecated':={'since',[46 | 48 | 53 | 54 | 101,...]}, 'importance':='hidden', 'required':='false', 'type':=_, 'validator':=fun((_) -> 'ok')}) breaks the contract (field_schema(), field_schema_map()) -> field_schema_fun()

We commonly use `deprecated => {since, "x.y.z"}`, but that is not what the typespec expects.

```
Line 72 Column 50: The call hocon_schema:override(Sc::atom() | fun((_) -> any()) | maybe_improper_list() | tuple() | map(), Override::#{'default':=[], 'deprecated':={'since',[46 | 48 | 53 | 54 | 101,...]}, 'importance':='hidden', 'required':='false', 'type':=_, 'validator':=fun((_) -> 'ok')}) breaks the contract (field_schema(), field_schema_map()) -> field_schema_fun()
```
@thalesmg thalesmg merged commit 97905ee into emqx:master Feb 27, 2024
3 checks passed
@thalesmg thalesmg deleted the fix-override-typespec branch February 27, 2024 19:56
@thalesmg thalesmg restored the fix-override-typespec branch February 27, 2024 19:56
@thalesmg thalesmg deleted the fix-override-typespec branch February 27, 2024 19:56
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