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

Warn about unused keys #12

Open
Zaharid opened this issue Nov 28, 2017 · 0 comments
Open

Warn about unused keys #12

Zaharid opened this issue Nov 28, 2017 · 0 comments

Comments

@Zaharid
Copy link
Contributor

Zaharid commented Nov 28, 2017

The semantics are trickier than it seems. We should detect whether the parent mapping is used as a namespace and if yes track every key inside. E.g. the root level is always a namespace but may contain a dict where we use the literal value.

However the identity comparison should be by input key rather than by nsspec.

This probably requires a much needed rewrite of the resolve_key machinery.

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

No branches or pull requests

1 participant