Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

remove old and unused Cody search page#63439

Merged
sqs merged 1 commit into
mainfrom
sqs/rm-old-cody-search
Jun 24, 2024
Merged

remove old and unused Cody search page#63439
sqs merged 1 commit into
mainfrom
sqs/rm-old-cody-search

Conversation

@sqs

@sqs sqs commented Jun 24, 2024

Copy link
Copy Markdown
Member

This page (at /cody/search) let you type in a natural-language description of what to search for, and then it used an LLM to rewrite that to Sourcegraph search query syntax. This was enabled using the cody-web-search feature flag, which has been disabled on dotcom for a while and was never documented or shared with customers, so it's safe to remove.

It was a cool idea, but we decided to focus on making Cody's search in the editor better, and we haven't seriously improved this for ~12 months.

Test plan

n/a; removed feature

@sqs sqs requested review from a team June 24, 2024 05:18
@cla-bot cla-bot Bot added the cla-signed label Jun 24, 2024
This page (at `/cody/search`) let you type in a natural-language description of what to search for, and then it used an LLM to rewrite that to Sourcegraph search query syntax. This was enabled using the `cody-web-search` feature flag, which has been disabled on dotcom for a while and was never documented or shared with customers, so it's safe to remove.

It was a cool idea, but we decided to focus on making Cody's search in the editor better, and we haven't seriously improved this for ~12 months.
@sqs sqs force-pushed the sqs/rm-old-cody-search branch from 1591f9e to 45fb3cb Compare June 24, 2024 06:15
@github-actions

Copy link
Copy Markdown
Contributor

Caution

License checking failed, please read: how to deal with third parties licensing.

@sqs sqs merged commit 673f9e8 into main Jun 24, 2024
@sqs sqs deleted the sqs/rm-old-cody-search branch June 24, 2024 15:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants