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

update pip install in injection example #305

Merged
merged 1 commit into from
May 16, 2024
Merged

Conversation

catel22
Copy link
Contributor

@catel22 catel22 commented May 16, 2024

Description

There were some issues with the pip install commands in notebook.

  • langkit install was not functional in the local VSCode environment

Changes

Considering this, changes were made:

  • Added quotes around 'langkit[all]' to ensure functional install in more environments

## Description

There were some issues with the pip install commands in notebook.

- langkit install was not functional in the local VSCode environment

## Changes

Considering this, changes were made:

- Added quotes around 'langkit[all]' to ensure functional install in more environments
Copy link
Collaborator

@jamie256 jamie256 left a comment

Choose a reason for hiding this comment

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

LGTM!

@catel22 catel22 merged commit d99aff3 into whylabs:main May 16, 2024
12 checks passed
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