Skip to content
This repository has been archived by the owner on Aug 12, 2023. It is now read-only.

feat(builtins): add gotests support in code actions #1362

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

joshrwolf
Copy link

Adds gotests as a builtin code action with 2 operating modes:

  1. package tests (available when cursor positioned over a package_clause)
  2. function test (available when cursor positioned over a identifier)

This PR is heavily influenced by the awesome work in #1339.

@joshrwolf joshrwolf force-pushed the gotests-code-action branch from bca4544 to 08c1425 Compare January 20, 2023 20:12
@jose-elias-alvarez
Copy link
Owner

This looks good, thanks! Can you go ahead and remove the extra print statements and also let me know how I can test this in case I have to investigate issues in the future?

@joshrwolf
Copy link
Author

thanks for the 👀! is there a designated place in the codebase for leaving testing information (like the meta.notes section?), or would you prefer it in this PR

@jose-elias-alvarez
Copy link
Owner

Just in this PR is fine - I found the approach in #1339 helpful to review the PR and then to test an additional change.

@ArtAndreev
Copy link
Contributor

ArtAndreev commented Jun 3, 2023

Can we get it merged, please? Also docs are needed to be updated, need fixing treesitter deprecations.

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.

3 participants