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

search contexts: remove auto-defined contexts graphql api#46083

Merged
limitedmage merged 2 commits into
mainfrom
jp/removeautodefined
Jan 3, 2023
Merged

search contexts: remove auto-defined contexts graphql api#46083
limitedmage merged 2 commits into
mainfrom
jp/removeautodefined

Conversation

@limitedmage

Copy link
Copy Markdown
Contributor

This is cleanup from #44903 which removed auto-defined contexts for all practical purposes.

Test plan

Verify all other context behavior is still as expected.

@cla-bot cla-bot Bot added the cla-signed label Jan 3, 2023
@limitedmage limitedmage requested review from a team, camdencheek and novoselrok January 3, 2023 17:27
@limitedmage limitedmage force-pushed the jp/removeautodefined branch from 3453d45 to 8fdc832 Compare January 3, 2023 17:28
@sourcegraph-bot

sourcegraph-bot commented Jan 3, 2023

Copy link
Copy Markdown
Contributor

Codenotify: Notifying subscribers in CODENOTIFY files for diff b3f69fe...f234b87.

Notify File(s)
@beyang internal/search/repos/repos_test.go
internal/search/searchcontexts/search_contexts.go
internal/search/searchcontexts/search_contexts_test.go
@camdencheek internal/search/repos/repos_test.go
internal/search/searchcontexts/search_contexts.go
internal/search/searchcontexts/search_contexts_test.go
@keegancsmith cmd/frontend/graphqlbackend/search_contexts.go
internal/search/repos/repos_test.go
internal/search/searchcontexts/search_contexts.go
internal/search/searchcontexts/search_contexts_test.go
@rvantonder cmd/frontend/graphqlbackend/search_contexts.go

@camdencheek camdencheek 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.

woot!

@limitedmage limitedmage merged commit 24297ed into main Jan 3, 2023
@limitedmage limitedmage deleted the jp/removeautodefined branch January 3, 2023 18:53
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.

3 participants