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

Examples for the accompanying blog post on search labs on how to use … #413

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

peter-strsr
Copy link

Code examples for Part 1 of the search labs article: https://github.com/elastic/search-labs-elastic-co/issues/1151

Copy link

gitnotebooks bot commented Mar 2, 2025

Found 1 changed notebook. Review the changes at https://app.gitnotebooks.com/elastic/elasticsearch-labs/pull/413

@JessicaGarson
Copy link
Contributor

JessicaGarson commented Mar 5, 2025

Hi, @peter-strsr,

It looks like the pre-commit checks are failing. You may want to try running the checks this way:

python -m venv .venv
.venv/bin/pip install -qqq -r requirements-dev.txt
.venv/bin/pre-commit install

You can find more infomation in our contributing guide. If you have issues after that, try moving the notebook to the supporting-blog-content folder, since this notebook corresponds to a blog post. This folder is more flexible when it comes to formatting checks.

@peter-strsr
Copy link
Author

Thanks for the help @JessicaGarson and sorry for missing the contribution guide!

I formatted the notebooks and moved them to the other folder.

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