Skip to content
This repository was archived by the owner on Apr 14, 2022. It is now read-only.

Allow navigation to stubs #1960

Merged
merged 42 commits into from
May 1, 2020
Merged

Conversation

MikhailArkhipov
Copy link

@MikhailArkhipov MikhailArkhipov commented Mar 20, 2020

No harm in showing stub for compiled code or when otherwise source is not available. Also fixes issue when documentation on os parts was not transferred since nt or posix are not submodules of os.

Closes #1921

MikhailArkhipov added 30 commits September 30, 2019 12:03
@jakebailey
Copy link
Member

Does this need a test for the linked issue? I'm surprised os.mkdir is compiled. os itself is required to be written in Python, so mkdir probably gets exposed via a different module, but I'd still be surprised that it's compiled to that extent.

@MikhailArkhipov
Copy link
Author

It is in nt which is compiled. Yeah, I can craft the test.

@jakebailey
Copy link
Member

The original issue was not on Windows, so hopefully the same behavior occurs there too.

@jakebailey
Copy link
Member

This appears to break the UpdateReferencesOnEdit test.

@jakebailey
Copy link
Member

jakebailey commented Apr 30, 2020

Specifically, the first Should gets only one reference rather than two; not sure which one is being dropped or if the old behavior was any good.

@MikhailArkhipov
Copy link
Author

Fixed, unintentional change.

Copy link
Member

@jakebailey jakebailey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, passes tests now. Would be good to look into jumping to scraper stubs in the future, but this fixes a few things.

@MikhailArkhipov MikhailArkhipov merged commit 2aef1f8 into microsoft:master May 1, 2020
@MikhailArkhipov MikhailArkhipov deleted the stubnav branch May 1, 2020 21:00
MikhailArkhipov pushed a commit that referenced this pull request Jun 15, 2020
* Remove stale reference

* Don't suppress LHS diagnostics on augmented assign

* Revert "Don't suppress LHS diagnostics on augmented assign"

This reverts commit 6109ac7.

* Escape [ and ]

* PR feedback

* Allow navigation to stubs

* Better handle stubs for compiled modules

* Fix typeshed path

* Partial undo

* Partial undo

* Undo accidental change
MikhailArkhipov pushed a commit that referenced this pull request Jun 15, 2020
* Remove stale reference

* Don't suppress LHS diagnostics on augmented assign

* Revert "Don't suppress LHS diagnostics on augmented assign"

This reverts commit 6109ac7.

* Escape [ and ]

* PR feedback

* Allow navigation to stubs

* Better handle stubs for compiled modules

* Fix typeshed path

* Partial undo

* Partial undo

* Undo accidental change
@dam5h dam5h mentioned this pull request Jul 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Go to Definition 'os.mkdir' error.
2 participants