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

Adds dap support #13

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

Conversation

florentcigolotti
Copy link

This is a starting point to add DAP support.

Current status

  • Test* functions
  • Sub test functions with t.Run
  • Example* functions
  • Benchmark functions (it is not supported by the plugin right now, so DAP will not work)
  • Testify suites

The status sign is broken right now after a debugging session, I haven't look at why right now.

@florentcigolotti
Copy link
Author

#12

@akinsho
Copy link
Collaborator

akinsho commented Aug 13, 2022

@florentcigolotti I think your work on this is a very good start. I'd suggest narrowing the scope of this PR though so it isn't blocked by these issues indefinitely. I think factor out stuff like subtests and testify to separate PRs since those aren't even super well-supported at all anyway.

I think the only real blocker here is getting the signs working. I think for those it's important to figure out what the status of each test is and make sure that test results are coming through in the format expected and the status value is correct.

@cedrata
Copy link

cedrata commented Oct 18, 2023

Are there any updates on this PR? Is someone still working on it?

@wwnbb
Copy link
Contributor

wwnbb commented Oct 29, 2023

Are there any updates on this PR? Is someone still working on it?

I created new pr, with lesser features.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants