-
Notifications
You must be signed in to change notification settings - Fork 7
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
testExecutableArgSingleCaseRegex is ignored #15
Comments
Thanks for finding this. Looks like I completely forgot to follow through on that change. Planning to get this working around the holiday period. |
Bump, just in case this has fallen off your radar. Thanks! |
still no news? |
Might be time for a fork, heh |
Hello! I'm starting work on #12 and trying to understand what has changed in vscode APIs in the last 2 years since I've touched this extension. I will take a look at this issue afterwards since for now I'm not even sure whether the extension works any more. I cannot say yet how long that will take since #12 may be pretty much a rewrite of the entire extension and I have very limited time. Any help would be appreciated! |
It still works - I'm using it! |
Implemented this as part of the switch to native vscode API. I thought that there's starting to be too many settings for this extension so I made the behavior hard coded instead of configurable. It will always pass |
Hello, it's been almost a year with no news from you so I'm closing this. If you still need something, feel free to reopen it. |
It was added to config and readme in 3e83088, but is never used.
Should be used here, but isn't: https://github.com/Florin-Popescu/vscode-unity-test-adapter/blob/master/src/adapter.ts#L427
Seems like a simple change, but I don't quite understand how it would get the test case name, as many of the functions are called with suite info only.
The text was updated successfully, but these errors were encountered: