vscode-dbt-power-user
vscode-dbt-power-user copied to clipboard
Remove deprecated apis
Overview
Problem
Describe the problem you are solving. Mention the ticket/issue if applicable.
Solution
Describe the implemented solution. Add external references if needed.
Screenshot/Demo
A picture is worth a thousand words. Please highlight the changes if applicable.
How to test
- Steps to be followed to verify the solution or code changes
- Mention if there is any settings configuration added/changed/deleted
Checklist
- [ ] I have run this code and it appears to resolve the stated issue
- [ ]
README.mdupdated and added information about my change
| :rocket: | This description was created by Ellipsis for commit 8eab166a494b0090d0a99d952d96066efe1d14b9 |
|---|
refactor: remove deprecated Python environment API usage
Summary:
Remove deprecated Python environment API usage by commenting out Python 3 checks and logging in VSCodeCommands, DBTClient, and PythonEnvironment.
Key points:
- Behavior:
- Commented out code in
VSCodeCommandsandDBTClientthat logs Python paths and checks for Python 3. - Removed
allPythonPathsandisPython3properties fromPythonEnvironment.
- Commented out code in
- Files Affected:
index.tsincommands: Commented out logging of Python paths.index.tsindbt_client: Commented out Python 3 check.pythonEnvironment.ts: RemovedallPythonPathsandisPython3properties.
Generated with :heart: by ellipsis.dev
Summary by CodeRabbit
-
Bug Fixes
- Disabled logging of Python paths to enhance security by preventing sensitive information from being printed.
- Simplified error handling by removing the check for Python 3 interpreter compatibility, allowing for smoother operation but potentially omitting important user prompts.
-
Refactor
- Streamlined the
PythonEnvironmentclass by removing unused properties and simplifying the constructor, focusing functionality on thedbtTerminal.
- Streamlined the