Skip to content

Commit 61352ec

Browse files
committed
Adjust comments and todos
1 parent f130906 commit 61352ec

File tree

2 files changed

+1
-1
lines changed

2 files changed

+1
-1
lines changed

src/validation/__tests__/OverlappingFieldsCanBeMergedRule-test.ts

-1
Original file line numberDiff line numberDiff line change
@@ -1024,7 +1024,6 @@ describe('Validate: Overlapping fields can be merged', () => {
10241024
it('does not infinite loop on immediately recursive fragment mentionned in queries', () => {
10251025
expectValid(`
10261026
query myQuery {
1027-
todoRemove
10281027
...fragA
10291028
}
10301029

src/validation/rules/OverlappingFieldsCanBeMergedRule.ts

+1
Original file line numberDiff line numberDiff line change
@@ -267,6 +267,7 @@ function collectConflictsBetweenFieldsAndFragment(
267267
// (E) Then collect any conflicts between the provided collection of fields
268268
// and any fragment names found in the given fragment.
269269
for (const referencedFragmentName of referencedFragmentNames) {
270+
// Don't compare this fragment with itself
270271
if (referencedFragmentName === fragmentName) {
271272
continue;
272273
}

0 commit comments

Comments
 (0)