f95d517a7c
Update rule content and descriptions to LaYC format. https://sonarsource.github.io/rspec/#/rspec/S4586 ## Review A dedicated reviewer checked the rule description successfully for: - [ ] logical errors and incorrect information - [ ] information gaps and missing content - [ ] text style and tone - [ ] PR summary and labels follow [the guidelines](https://github.com/SonarSource/rspec/#to-modify-an-existing-rule)
36 lines
871 B
Plaintext
36 lines
871 B
Plaintext
|
|
ifdef::env-github,rspecator-view[]
|
|
|
|
'''
|
|
== Implementation Specification
|
|
(visible only on this page)
|
|
|
|
=== Message
|
|
|
|
Do not return null from this method, instead return Task.FromResult<T>(null).
|
|
|
|
=== Highlighting
|
|
|
|
return null
|
|
|
|
'''
|
|
== Comments And Links
|
|
(visible only on this page)
|
|
|
|
=== on 16 Apr 2018, 15:00:41 Amaury Levé wrote:
|
|
\[~ann.campbell.2] Could you please review?
|
|
|
|
=== on 17 Apr 2018, 21:43:26 Ann Campbell wrote:
|
|
\[~amaury.leve] I'm confused. The title says "in a method *not* marked as async" and the description talks about "Returning ``++null++`` from a *non*-``++async++``".
|
|
|
|
|
|
Also, this title doesn't follow the format: X should [not] Y
|
|
|
|
=== on 18 Apr 2018, 09:30:40 Amaury Levé wrote:
|
|
Thank you for the review. Does it looks better now?
|
|
|
|
=== on 18 Apr 2018, 13:43:19 Ann Campbell wrote:
|
|
Yes, looks good [~amaury.leve]
|
|
|
|
endif::env-github,rspecator-view[]
|