Skip to content

Conversation

@Kalmaegi
Copy link
Contributor

No description provided.

@Kalmaegi
Copy link
Contributor Author

@connor4312 I ran into some confusion, I controlled the conditions of the appearance, but it didn't seem to work, and the content still showed up on editor menu of each file

@connor4312
Copy link
Member

Thanks for the PR! There's more work needed since the capabilities are global and we didn't previously have any variable that applied just to the active editor. We also needed work to make the lookup efficient, since previously navigating to a file where there were no tests would result in VS Code iterating over the entire test collection.

Since there were a lot more changes needed, I've extended this and tackled them in #167091. Although this PR didn't get merged, we really appreciate your contribution 🙂

@connor4312 connor4312 closed this Nov 23, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Jan 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants