Skip to content

Use unknown instead of any where possible#3343

Merged
robertbrignull merged 4 commits intomainfrom
robertbrignull/use-unknown
Feb 12, 2024
Merged

Use unknown instead of any where possible#3343
robertbrignull merged 4 commits intomainfrom
robertbrignull/use-unknown

Conversation

@robertbrignull
Copy link
Copy Markdown
Contributor

This is all the cases I could find where we are using any but we can just replace it with unknown without changing anything else. If everything still compiles then this should be safe to do so.

Checklist

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

@robertbrignull robertbrignull requested a review from a team February 12, 2024 15:40
@robertbrignull robertbrignull requested a review from a team as a code owner February 12, 2024 15:40
@robertbrignull robertbrignull merged commit 5a80d02 into main Feb 12, 2024
@robertbrignull robertbrignull deleted the robertbrignull/use-unknown branch February 12, 2024 15: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