-
Notifications
You must be signed in to change notification settings - Fork 293
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
Run above should run all code above, not just cells. #2753
Comments
@gaetawoo, this issue was created in response to the discussion you had with @rchiodo in microsoft/vscode-python#8392. While this is a valid request, we have decided not to change this behavior. Why? Well, we have already shipped this way for a while. Therefore, if we change this, we could be affecting users who happened to find the current behavior satisfactory, or even desired. Given the problem is likely discoverable and easily worked around, as well, we just don't want to change it. We really appreciate your feedback though! |
@greazer I wouldn't say it's really discoverable at all. And while it might be easy to work around, there is no indication as to why all code above doesn't run. You'd have to know a priori that only above cells will run. Maybe the label should be changed to be more precise. The only way I found out what the issue was was by chatting with a maintainer on a GitHub issue. |
Validated. |
In fact why does it need to run them as cells in the first place?
Related to issue microsoft/vscode-python#8392
The text was updated successfully, but these errors were encountered: