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

Super-fy the Search Expressions doc #5368

Merged
merged 2 commits into from
Oct 26, 2024
Merged

Super-fy the Search Expressions doc #5368

merged 2 commits into from
Oct 26, 2024

Conversation

philrz
Copy link
Contributor

@philrz philrz commented Oct 25, 2024

What's Changing

This is a first pass at adapting the Search Expressions doc for the "super" era.

Details

It's expected all docs pages will have to be somewhat revised to ease out the "Zed era" terminology and make way for the "SuperDB era". In an attempt to chip away at them incrementally, I'm starting from the pages hyperlinked from the revised README (#5367) and will fan out from there. I expect these may need more passes when SuperSQL is further along since right now the examples are all pure SuperPipe since that's what's working in main.

Like other early docs rewrite attempts, rather than replacing references to "Zed" with 1-for-1 references to something "super", I feel like I'm having some success with reworking sentences to drop the reference entirely.

Since I had to review everything, I made a few additional fixes and added some polish along the way.

@@ -205,7 +207,7 @@ where grep("foo", this)
```

Note that the "search" keyword may be omitted.
For example, the simplest Zed program is perhaps a single keyword search, e.g.,
For example, the simplest SuperPipe program is perhaps a single keyword search, e.g.,
Copy link
Collaborator

Choose a reason for hiding this comment

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

program -> query

@philrz philrz merged commit 1d85212 into main Oct 26, 2024
4 checks passed
@philrz philrz deleted the docs-search-expr-super branch October 26, 2024 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants