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
Copy link
Copy Markdown
Contributor

@medb medb commented Apr 6, 2026

No description provided.

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

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

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