UnitsNet icon indicating copy to clipboard operation
UnitsNet copied to clipboard

Wrong Plural for MeterPerMinute Speed Units Names

Open Markussp256 opened this issue 2 years ago • 1 comments

Describe the bug For Speed we have many Units of the form [LengthUnit]Per[TimeUnit]. For instance MeterPerSecond. The plural of it we get by taking the plural of the Lengthunit and the singular of the TimeUnit, for instance MetersPerSecond. However for Units with Meter and Minute also the TimeUnit is currently in Plural form. This makes the method names somehow inconsistent.

Speed has for example the static method FromMetersPerMinutes and FromInchesPerMinute. This also affects all corresponding Units with a prefix, e.g. NanoMetersPerMinutes.

Expected behavior I would appreciate new methods without the s. The current methods with s should still be usable for a while to not break any code but with a warning that it will be deprecated.

Markussp256 avatar Nov 19 '23 15:11 Markussp256

Thanks for reporting, could you please post a list of the units with wrong plural form on #1200 ?

angularsen avatar Nov 24 '23 23:11 angularsen

Fixed by #1351 in v6

angularsen avatar Jul 08 '24 13:07 angularsen