Skip to content

feat: Auto-detect more popular IDEs for usage tracking - #182

Merged
medb merged 3 commits into
mainfrom
ide_tracking
Apr 6, 2026
Merged

feat: Auto-detect more popular IDEs for usage tracking#182
medb merged 3 commits into
mainfrom
ide_tracking

Conversation

@medb

@medb medb commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@medb
medb requested review from tim-u and virrrat April 6, 2026 18:46

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds environment detection for multiple platforms including Kaggle, Databricks, and SageMaker, while updating the environment labeling priority. Review feedback suggests simplifying the Eclipse detection logic to remove redundancy and adjusting the interactive flag check to support multiple levels of interactivity. Additionally, refactoring the unit tests is recommended to improve maintainability by reducing the number of individual mock decorators.

Comment thread google/cloud/dataproc_spark_connect/environment.py
Comment thread google/cloud/dataproc_spark_connect/environment.py Outdated
Comment thread tests/unit/test_environment.py
medb and others added 2 commits April 6, 2026 12:48
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@medb
medb merged commit 51c6f92 into main Apr 6, 2026
6 checks passed
@medb
medb deleted the ide_tracking branch April 6, 2026 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants