-
Notifications
You must be signed in to change notification settings - Fork 27.5k
Add MISSING_TYPE_LINTER to detect new, untyped symbols (fix #163990) #167075
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
Open
rec
wants to merge
43
commits into
gh/rec/167/base
Choose a base branch
from
gh/rec/167/head
base: gh/rec/167/base
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
fcd752f
Update
rec a327cdb
Update
rec 43df98d
Update
rec 9d6b99b
Update
rec d11ba14
Update
rec f79b7ff
Update
rec e21f95d
Update
rec 591b2df
Update
rec 31a93ac
Update
rec ca7e2db
Update
rec acb37b8
Update
rec df9873c
Update
rec b15e366
Update
rec 5e932ce
Update
rec 71892fe
Update
rec dc8e47d
Update
rec a8bf1b1
Update
rec 8081ce7
Update
rec 4699383
Update
rec 885db95
Update
rec 92fff67
Update
rec 54f5cff
Update
rec f89b55f
Update
rec 31b8697
Update
rec b90be99
Update
rec 4b021ac
Update
rec d6bb082
Update
rec 246227a
Update
rec 12367eb
Update
rec 1e0c7db
Update
rec 11cfd4b
Update
rec 402f798
Update
rec b114c0a
Update
rec 99adf63
Update
rec c57b323
Update
rec 01b7db1
Update
rec ea3fa54
Update
rec 58d5ac4
Update
rec e268603
Update
rec 9134ec0
Update
rec dfc87cf
Update
rec 271731d
Update
rec 851cc4e
Update
rec File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we fix this function before landing? Otherwise what this linter consider public is not aligned with what other tests consider public.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, direct me to the official definition and I'll use it in a trice!