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

Silence errors when not on a Kedro project #154

Merged

Conversation

SajidAlamQB
Copy link
Contributor

@SajidAlamQB SajidAlamQB commented Nov 15, 2024

Related to: #77

Uers were experiencing error messages from the Kedro VSCode extension when working in non-Kedro projects. These errors were causing annoyance and could potentially lead users to uninstall the extension due to the unnecessary noise.

Development Notes:

Introduced a check in the activate function within extension.ts to determine if the current workspace is a Kedro project.
The check looks for a pyproject.toml file containing the [tool.kedro] section.
If the workspace is not a Kedro project, the extension will not activate.

Added isKedroProject function from in utilities to perform the project detection logic.

Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
@SajidAlamQB SajidAlamQB requested a review from noklam November 20, 2024 15:11
@SajidAlamQB SajidAlamQB marked this pull request as ready for review November 20, 2024 15:11
Copy link
Contributor

@noklam noklam left a comment

Choose a reason for hiding this comment

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

I have tested and it works as expected, left some comments mainly about refactoring with existing function and consider some edge cases.

Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
@SajidAlamQB SajidAlamQB requested a review from jitu5 November 21, 2024 14:04
@noklam noklam self-requested a review November 22, 2024 08:49
Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
@SajidAlamQB SajidAlamQB merged commit d37d02d into main Nov 22, 2024
2 checks passed
@SajidAlamQB SajidAlamQB deleted the bugfix/Silence-vscode-errors-when-not-on-a-Kedro-project branch November 22, 2024 11:52
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.

3 participants