artin/sourcegraph
1
0
Fork 0
mirror of https://github.com/sourcegraph/sourcegraph.git synced 2026-02-06 17:51:57 +00:00
Code Issues Packages Projects Releases Wiki Activity

codeintel: make usage-range non-optional (#64236)

Browse Source 
Closes
https://linear.app/sourcegraph/issue/GRAPH-727/make-range-for-usagerange-non-optional

## Test plan

Tests continue to pass
This commit is contained in:
Christoph Hegemann 2024-08-02 16:51:57 +02:00 committed by GitHub
parent 4a565a8819
commit 7fc92a5538
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cmd/frontend/graphqlbackend/codeintel.codenav.graphql
View File

@ -746,7 +746,7 @@ type Usage {
# Invariant: `range.path` == `blob.path`. The path is made available
# as part of this type for convenience.
# """
usageRange: UsageRange
usageRange: UsageRange!
# TODO: Add a blob: GitBlob field here for more flexibility?

4
internal/codeintel/codenav/transport/graphql/root_resolver_usages.go
View File

@ -179,8 +179,8 @@ func (u *usageResolver) DataSource() *string {
panic("implement me")
}
func (u *usageResolver) UsageRange(ctx context.Context) (resolverstubs.UsageRangeResolver, error) {
return u.usageRange, nil
func (u *usageResolver) UsageRange(ctx context.Context) resolverstubs.UsageRangeResolver {
return u.usageRange
}
func (u *usageResolver) SurroundingContent(ctx context.Context) string {

2
internal/codeintel/resolvers/codenav.go
View File

@ -533,7 +533,7 @@ type UsageResolver interface {
Symbol(context.Context) (SymbolInformationResolver, error)
Provenance(context.Context) (codenav.CodeGraphDataProvenance, error)
DataSource() *string
UsageRange(context.Context) (UsageRangeResolver, error)
UsageRange(context.Context) UsageRangeResolver
SurroundingContent(_ context.Context) string
UsageKind() SymbolUsageKind
}