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

Update README.md to reflect state of native UI #233

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JDaance
Copy link

@JDaance JDaance commented Apr 4, 2022

This PR is a bit tongue in cheek, I spent quite a bit of time migrating to the native version, in part due to this text of deprecation. The native version is not the same as this extension and more, its different, is currently missing support for autorun, and I really missed the scroll bar indicators :). If I would be blunt I would say that the native version is in a beta state, and I had many perceived performance problems, but opinions differ. I hope this extension lives on!

For reference, this is a list of my filed tickets on the native version: https://github.com/microsoft/vscode/issues?q=author%3Ajdaance+
And the general ticket on missing auto run: microsoft/vscode#134941

I did not file all my complaints about it, I just tried to work with it until I gave up and reverted back to this extension.

@hbenl
Copy link
Owner

hbenl commented Apr 10, 2022

I'm actually considering removing the deprecation notice because after initially having high hopes for native testing I'm now fairly disappointed by the state in which Microsoft left it.
FWIW the feature that I am missing is "retiring" tests, i.e. when you change a source file after running tests, the test results will be shown grayed out, so you can see what the last results were but also that they are outdated. I have some test suites that take a long time to run and this feature is very important to me in that situation.

I hope this extension lives on!

Don't worry, it will!

Copy link

@ralfhandl ralfhandl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The hint on autorun would have saved me some time, and it is why I switched back to the old UI.

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.

3 participants