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

Smart Send default value should be true #23074

Merged
merged 11 commits into from
Mar 14, 2024

Conversation

anthonykim1
Copy link

@anthonykim1 anthonykim1 commented Mar 14, 2024

Forgot to set default value to true here: #23067
It was true before removing experiment code since we had experiment to 100%, which effectively overrode the smart send setting value to True.

@anthonykim1 anthonykim1 added feature-request Request for new features or functionality area-repl labels Mar 14, 2024
@anthonykim1 anthonykim1 self-assigned this Mar 14, 2024
@anthonykim1 anthonykim1 marked this pull request as ready for review March 14, 2024 20:43
@anthonykim1 anthonykim1 added the skip tests Updates to tests unnecessary label Mar 14, 2024
@VSCodeTriageBot VSCodeTriageBot added this to the March 2024 milestone Mar 14, 2024
@anthonykim1 anthonykim1 marked this pull request as draft March 14, 2024 20:56
@anthonykim1 anthonykim1 marked this pull request as ready for review March 14, 2024 22:36
@anthonykim1 anthonykim1 merged commit 028398e into microsoft:main Mar 14, 2024
35 checks passed
wesm pushed a commit to posit-dev/positron that referenced this pull request Apr 5, 2024
Forgot to set default value to true here:
microsoft/vscode-python#23067
It was true before removing experiment code since we had experiment to
100%, which effectively overrode the smart send setting value to True.
wesm pushed a commit to posit-dev/positron that referenced this pull request Apr 8, 2024
Forgot to set default value to true here:
microsoft/vscode-python#23067
It was true before removing experiment code since we had experiment to
100%, which effectively overrode the smart send setting value to True.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-repl feature-request Request for new features or functionality skip tests Updates to tests unnecessary
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants