Skip to content
This repository was archived by the owner on Jun 13, 2025. It is now read-only.

Commit a8b0011

Browse files
Remove criticalFiles from graphQL API
This is not being used in Gazebo, so removing it.
1 parent 0566c19 commit a8b0011

File tree

5 files changed

+0
-19
lines changed

5 files changed

+0
-19
lines changed

graphql_api/tests/test_repository.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,6 @@
6161
defaultBranch
6262
author { username }
6363
profilingToken
64-
criticalFiles { name }
6564
graphToken
6665
yaml
6766
isATSConfigured
@@ -138,7 +137,6 @@ def test_when_repository_has_no_coverage(self):
138137
"defaultBranch": "main",
139138
"author": {"username": "codecov-user"},
140139
"profilingToken": "",
141-
"criticalFiles": [],
142140
"graphToken": graphToken,
143141
"yaml": "test: test\n",
144142
"isATSConfigured": False,
@@ -201,7 +199,6 @@ def test_when_repository_has_coverage(self):
201199
"defaultBranch": "main",
202200
"author": {"username": "codecov-user"},
203201
"profilingToken": "",
204-
"criticalFiles": [],
205202
"graphToken": graphToken,
206203
"yaml": "test: test\n",
207204
"isATSConfigured": False,

graphql_api/types/commit/commit.graphql

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ type Commit {
1717
last: Int
1818
before: String
1919
): UploadConnection @cost(complexity: 10, multipliers: ["first", "last"])
20-
criticalFiles: [CriticalFile!]!
2120
pathContents(path: String, filters: PathContentsFilters): PathContentsResult
2221
deprecatedPathContents(path: String, filters: PathContentsFilters, first: Int, after: String, last: Int, before: String): DeprecatedPathContentsResult
2322
errors(errorType: CommitErrorType!): CommitErrorsConnection!

graphql_api/types/commit/commit.py

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -133,13 +133,6 @@ async def resolve_compare_with_parent(commit: Commit, info, **kwargs):
133133
return ComparisonReport(commit_comparison)
134134

135135

136-
@commit_bindable.field("criticalFiles")
137-
@sync_to_async
138-
def resolve_critical_files(commit: Commit, info, **kwargs) -> List:
139-
"""DEPRECATED. Returning empty list."""
140-
return []
141-
142-
143136
@sentry_sdk.trace
144137
def get_sorted_path_contents(
145138
current_owner: Owner,

graphql_api/types/repository/repository.graphql

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ type Repository {
4040
): BranchConnection @cost(complexity: 3, multipliers: ["first", "last"])
4141
defaultBranch: String
4242
profilingToken: String
43-
criticalFiles: [CriticalFile!]!
4443
graphToken: String
4544
yaml: String
4645
bot: Owner

graphql_api/types/repository/repository.py

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -179,13 +179,6 @@ def resolve_static_analysis_token(
179179
return command.get_repository_token(repository, token_type="static_analysis")
180180

181181

182-
@repository_bindable.field("criticalFiles")
183-
@sync_to_async
184-
def resolve_critical_files(repository: Repository, info: GraphQLResolveInfo) -> List:
185-
"""Deprecated. Return dummy."""
186-
return []
187-
188-
189182
@repository_bindable.field("graphToken")
190183
def resolve_graph_token(repository: Repository, info: GraphQLResolveInfo) -> str:
191184
return repository.image_token

0 commit comments

Comments
 (0)