Conversation
paramterless => parameterless Polymorhic => Polymorphic
eerhardt
requested review from
CoffeeFlux,
ahsonkhan,
eiriktsarpalis,
jozkee,
lambdageek,
layomia,
marek-safar,
steveharter,
thaystg and
vargaz
as code owners
|
Tagging subscribers to this area: @CoffeeFlux Issue Detailsparamterless => parameterless
|
| { | ||
| PublicParameterizedConstructorTestClass paramterless = PublicParameterizedConstructorTestClass.Instance; | ||
| Assert.Equal("{\"Name\":\"42\"}", JsonSerializer.Serialize(paramterless)); | ||
| PublicParameterizedConstructorTestClass parameterless = PublicParameterizedConstructorTestClass.Instance; |
There was a problem hiding this comment.
How did you detect these or did you happen to just run across them ad-hoc?
There was a problem hiding this comment.
Funny story - my tests were broken in #45911, and I noticed that the exception message coming from mono had this spelling issue. So I grepped the whole repo for the same spelling error and this came up.
Then my eye caught the "Polymorphic" spelling mistake below, so I figured I had to fix that as well, while I was in here.
There was a problem hiding this comment.
Ah, good old grepping for a string with a typo :)
ghost
locked as resolved and limited conversation to collaborators
3 participants
paramterless => parameterless
Polymorhic => Polymorphic