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

Test: Warning message and setting for invalid Smart Send #22536

Closed
3 tasks done
anthonykim1 opened this issue Nov 27, 2023 · 0 comments
Closed
3 tasks done

Test: Warning message and setting for invalid Smart Send #22536

anthonykim1 opened this issue Nov 27, 2023 · 0 comments

Comments

@anthonykim1
Copy link

anthonykim1 commented Nov 27, 2023

Refs: #22340

Complexity: 3

Create Issue


Thank you for testing warning message and setting for smart send!

Background Information

In some scenarios, users have deprecated Python code (such as Python 2) code or other language code (such as Rust) in their Python file. Python AST that is used for Smart Send logic (where we smartly figure out minimum executable code in a given Python file) fails to parse and compute the logic for deprecated and invalid Python code.

In that case, we would want to let user know that Python is unable to parse their currently opened Python file and hence unable to use smart send feature.
We also want to provide readily accessible button that user can press to simply disable the Smart Send feature, and provide option for them to change that in the extension user&workspace setting.

Setup:

  1. Make sure to have latest version of Python extension installed.
  2. Make sure you have Python in your computer

Steps

  1. Open or create a Python file
  2. Try writing some deprecated, invalid Python code or inserting some other language code in your Python file.
  3. Shift+enter and see if you get the warning message and a button to disable Smart Send
  4. Press on the button and shift+enter again to see if that disabled Smart Send

Description on what Smart Send does is provided here: #21778 and #21838

Thank you for testing!

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

No branches or pull requests

4 participants