FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix telemetry not getting picked up for Terminal REPL by anthonykim1 · Pull Request #23958 · microsoft/vscode-python · GitHub

Fix telemetry not getting picked up for Terminal REPL - #23958

Merged
Anthony Kim (anthonykim1) merged 3 commits into
microsoft:mainfrom
anthonykim1:fixTelemetry
Aug 16, 2024
Merged

Anthony Kim (anthonykim1) merged 3 commits into
microsoft:mainfrom
anthonykim1:fixTelemetry

Conversation

Anthony Kim (anthonykim1) commented Aug 16, 2024
edited
Loading

Copy link
Copy Markdown

#23941
Having no blank spaces in front of plain 'python' or 'python3' were returning falsey value.

Also comply to new ruff rule.

Anthony Kim (anthonykim1) added area-repl bug Issue identified by VS Code Team member as probable bug skip tests Updates to tests unnecessary labels Aug 16, 2024
Anthony Kim (anthonykim1) marked this pull request as ready for review August 16, 2024 20:17
vs-code-engineering Bot added this to the August 2024 milestone Aug 16, 2024
Anthony Kim (anthonykim1) enabled auto-merge (squash) August 16, 2024 20:33
Anthony Kim (anthonykim1) marked this pull request as draft August 16, 2024 20:39
auto-merge was automatically disabled August 16, 2024 20:39

Pull request was converted to draft

Anthony Kim (anthonykim1) marked this pull request as ready for review August 16, 2024 20:56
Anthony Kim (anthonykim1) enabled auto-merge (squash) August 16, 2024 20:56
Anthony Kim (anthonykim1) merged commit e9f1071 into microsoft:main Aug 16, 2024
Anthony Kim (anthonykim1) added a commit to anthonykim1/vscode-python that referenced this pull request Sep 13, 2024
microsoft#23941 
Having no blank spaces in front of plain 'python' or 'python3' were
returning falsey value.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-repl bug Issue identified by VS Code Team member as probable bug skip tests Updates to tests unnecessary

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL