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

Try shell integration proposed API for Python extension #23769

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

Conversation

anthonykim1
Copy link

@anthonykim1 anthonykim1 commented Jul 8, 2024

DO NOT MERGE
Reference: #23738

As of Jul 9th: struggling to get exit code beside undefined. Was able to get exit code of 0 for once, and continuing to get undefined. Starting out with lightweight test where I would basically create terminal, send dummy command "echo hello world" and trying to observe the exit code of this.

TODO: Later should be creating hidden terminal that will print out user's environment variable living under their shell specific initialization scripts.
TODO: Also figure out observing whether terminal REPL successfully by using shell integration proposed API instead of terminal data write event.
TODO: Also figure out which additional activation commands we need to test out for Python extension.

@anthonykim1 anthonykim1 added area-terminal area-environments Features relating to handling interpreter environments labels Jul 8, 2024
@anthonykim1 anthonykim1 self-assigned this Jul 8, 2024
@anthonykim1 anthonykim1 changed the title Try shell integration API for Python extension Try shell integration proposed API for Python extension Jul 9, 2024
@anthonykim1
Copy link
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-environments Features relating to handling interpreter environments area-terminal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant