Skip to content

Remove empty cell from 11_advanced_queries.ipynb#584

Merged
vishal-bala merged 1 commit intoredis:mainfrom
paoloredis:patch-1
Apr 17, 2026
Merged

Remove empty cell from 11_advanced_queries.ipynb#584
vishal-bala merged 1 commit intoredis:mainfrom
paoloredis:patch-1

Conversation

@paoloredis
Copy link
Copy Markdown
Contributor

@paoloredis paoloredis commented Apr 15, 2026

This empty cell breaks the redisvl automation that generates the markdown documentation


Note

Low Risk
Low risk documentation-only change that only deletes a stray notebook cell and does not affect library/runtime behavior.

Overview
Removes a stray empty markdown cell (containing only ##) from docs/user_guide/11_advanced_queries.ipynb, preventing the notebook-to-markdown documentation automation from breaking.

Reviewed by Cursor Bugbot for commit b0be9c9. Bugbot is set up for automated code reviews on this repo. Configure here.

@vishal-bala vishal-bala added the auto:documentation Changes only affect the documentation label Apr 15, 2026
@paoloredis
Copy link
Copy Markdown
Contributor Author

Would it be possible to patch releases with this fix all the way back to 0.13.0?

@vishal-bala
Copy link
Copy Markdown
Collaborator

Hey @paoloredis, that sounds like a bit of a headache 😅
How does the documentation workflow run? Is it based off of a new RedisVL tag being published? We could possibly look at patching just the latest patch version for each minor release.

@paoloredis
Copy link
Copy Markdown
Contributor Author

Hi @vishal-bala , the workflow runs on a daily schedule and it pulls in all tags up until 0.6.0. it generates documentation for each tag

@vishal-bala
Copy link
Copy Markdown
Collaborator

Gotcha 👍 I'll talk to the team about what we can do to get those patched, it sounds like it might be a hairy process. Is there any possibility to have the documentation workflow filter out the empty cell on that end?

@vishal-bala vishal-bala merged commit 7cafaa0 into redis:main Apr 17, 2026
11 checks passed
@paoloredis
Copy link
Copy Markdown
Contributor Author

I ended up patching this on my end

@applied-ai-release-bot
Copy link
Copy Markdown

🚀 PR was released in v0.18.0 🚀

@applied-ai-release-bot applied-ai-release-bot Bot added the released This issue/pull request has been released. label Apr 21, 2026
@vishal-bala
Copy link
Copy Markdown
Collaborator

@paoloredis Awesome! Thanks for sorting that out 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto:documentation Changes only affect the documentation released This issue/pull request has been released.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants