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

Add support for detecting tests in source files, and implement it for Rust #11195

Merged
merged 31 commits into from
May 5, 2024

Conversation

osiewicz
Copy link
Contributor

Continuing work from #10873

Release Notes:

  • TODO

mikayla-maki and others added 23 commits April 25, 2024 16:52
Restore live updating language queries
Co-authored-by: Mikayla <mikayla@zed.dev>
- With OG code actions, check line number of active actions menu before deciding whether to use 'selected' color for code actions bolt (fixes: wrong color of bolt when both test indicator and code action bolt is visible)
- Fixed position of code actions menu when clicking on not-yet-selected line (it is currently attached to current line).
- Fixed usage of cmd-. with code actions (we did not wire through row number properly prior to that and tasks didn't show up when opening the context menu through keybind.
- Wire through selected status for task indicator.
Additionally, fix a bug where the task menu would pick up code actions from currently focused line, even when it may not be applicable to that task menu.
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Apr 30, 2024
@osiewicz
Copy link
Contributor Author

osiewicz commented May 2, 2024

I believe all that's left before we land this is support for tag rebinding (allowing users to bind given tag to a different task); I plan to merge this PR as is optimistically on Sunday to have it in Nightly by Monday and then move with implementing task rebinding.

Also, Mikayla remarked that we should not be showing test indicators in remote projects.

@osiewicz osiewicz merged commit 5a71d8c into main May 5, 2024
8 checks passed
@osiewicz osiewicz deleted the testing-infra-code-actions-change branch May 5, 2024 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants