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

Report unused properties based on binding definition #808

Closed
vordimous opened this issue Feb 21, 2024 · 0 comments · Fixed by #929
Closed

Report unused properties based on binding definition #808

vordimous opened this issue Feb 21, 2024 · 0 comments · Fixed by #929
Assignees
Labels
breaking Backwards incompatible story Feature description from user's perspective

Comments

@vordimous
Copy link
Contributor

vordimous commented Feb 21, 2024

Describe the desired outcome from the user's perspective
As a user editing a zilla config, I would like to get validation feedback for any valid property definitions that are not used or invalid based on other parts of the config. It is difficult to self diagnose issues when the config is visibly valid but the functionality isn't.

Acceptance criteria

  • When a kind of binding determines which properties are needed or ignored, validate the properties and report to the user.

Additional context
One example is between the cache_server and cache_client kinds of kafka binding. cache_server can have bootstrap topics and cache_client does not need it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Backwards incompatible story Feature description from user's perspective
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants