Roman Ivantsov

Results 120 comments of Roman Ivantsov

no, not yet. working on it

Since the new style guide sets the Title case for headings, this PR fixes the remaining non-conforming headers to Title case, so it is ready to merge.

Matt's suggestion: > Supporting subscriptions requires a different approach to queries and mutations. I am still not quite happy with this. Is it "to queries and mutations" or "COMPARED to...

@benjie , can you pls help with this 'code style/pretier' failure, have no idea what to do about it

Closing this PR the first part (mutation/subscr ambiguity) is fixed by another PR. I still think the second paragraph (Likewise...) is extremely confusing, but I am out of arguments here.

I think there's some confusion here. The original text gives a (wrong) idea that merge/not merge may depend on parallel/serial execution mode chosen by the server - which it free...

I understand you try to use 'if can be done in parallel' as criteria for 'should be merged', but I think this brings more confusion. I think parallel should be...

I looked at other places where Field merge is discussed and I think there's more wordings to fix there. I am closing this PR to open a discussion first and...

@jsshapiro , agree 100%. Or maybe 99. I tried to push these issues but gave up, it's no use. The guys who make decisions there are absolutely against ANY change....