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

Conversation

@emidoots
Copy link
Member

This will be used both from the GraphQL resolver layer, as well as the
background workers - both of which need to scan the user/org/global settings
for insights defined within.

Also @efritz this cleans up the settings store mocks :)

Stephen Gutekanst added 2 commits February 12, 2021 18:01
This will be used both from the GraphQL resolver layer, as well as the
background workers - both of which need to scan the user/org/global settings
for insights defined within.

Signed-off-by: Stephen Gutekanst <stephen@sourcegraph.com>
Signed-off-by: Stephen Gutekanst <stephen@sourcegraph.com>
@emidoots emidoots added team/internal-contributor code-insights Issues related to the Code Insights product labels Feb 13, 2021
@emidoots emidoots added this to the 3.25 milestone Feb 13, 2021
@emidoots emidoots requested a review from efritz February 13, 2021 01:08
@emidoots emidoots mentioned this pull request Feb 13, 2021
27 tasks
@sourcegraph-bot
Copy link
Contributor

sourcegraph-bot commented Feb 13, 2021

Notifying subscribers in CODENOTIFY files for diff a2311ca...bd4f455.

Notify File(s)
@felixfbecker enterprise/internal/insights/discovery/discovery.go
enterprise/internal/insights/discovery/discovery_test.go
enterprise/internal/insights/discovery/gen.go
enterprise/internal/insights/discovery/mock_setting_store.go
enterprise/internal/insights/resolvers/insight_connection_resolver.go
enterprise/internal/insights/resolvers/insight_connection_resolver_test.go
enterprise/internal/insights/resolvers/insight_series_resolver_test.go

Signed-off-by: Stephen Gutekanst <stephen@sourcegraph.com>
@emidoots emidoots merged commit 8176ccd into main Feb 15, 2021
@emidoots emidoots deleted the sg/insights-discovery branch February 15, 2021 20:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

code-insights Issues related to the Code Insights product

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants