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

Remove any from PermissionController types #4237

Open
3 tasks
rekmarks opened this issue May 1, 2024 · 0 comments
Open
3 tasks

Remove any from PermissionController types #4237

rekmarks opened this issue May 1, 2024 · 0 comments

Comments

@rekmarks
Copy link
Member

rekmarks commented May 1, 2024

Following #4171, any is only used in a small number of PermissionController types. They are mostly, but not always used as constraints. We should eliminate any from the following types:

For the *SpecificationBase types, I believe the solution is to genericize them such that TypeScript can determine whether a concrete specification is valid. The refactor necessary to accomplish this may not be terrible, but it's not trivial.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants