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

Rename intent API (add chat reference, make the name more idiomatic)#63416

Merged
rafax merged 3 commits into
mainfrom
rg/rename_intent_api
Jun 28, 2024
Merged

Rename intent API (add chat reference, make the name more idiomatic)#63416
rafax merged 3 commits into
mainfrom
rg/rename_intent_api

Conversation

@rafax

@rafax rafax commented Jun 21, 2024

Copy link
Copy Markdown
Contributor

Address remarks from https://github.com/sourcegraph/sourcegraph/pull/63228#issuecomment-2163682435

Test plan

  • tested locally
  • old code not deleted yet (until I confirm it's ok to remove it)

@cla-bot cla-bot Bot added the cla-signed label Jun 21, 2024
@rafax rafax requested review from a team and sqs June 21, 2024 09:34
): [CodyContextResult!]!
"""
EXPERIMENTAL: Detect intent for a given Cody query.
DEPRECATED: use chatIntent. This will be deleted in a future release.

@rafax rafax Jun 21, 2024

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@sqs are you using this anywhere? I don't see it being used on S2, if it is unused I can remove it it in this PR.

@sqs sqs left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@rafax rafax enabled auto-merge (squash) June 28, 2024 11:25
@rafax rafax merged commit 131c1b5 into main Jun 28, 2024
@rafax rafax deleted the rg/rename_intent_api branch June 28, 2024 11:55
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