Fix wrong SolarMass abbreviation

[Brunni]

The wrong UTF-8 symbol was used.

Correct: https://en.wiktionary.org/wiki/%E2%98%89

Wrong:
https://en.wikipedia.org/wiki/Circled_dot

[tmilnthorp]

Interesting! This would be a breaking parse change. We should leave the old incorrect string as an abbreviation (non default), so that parsing the incorrect symbol will work until a major version. Can we mark abbreviation strings as obsolete @angularsen?

[angularsen]

Interesting! This would be a breaking parse change. We should leave the old incorrect string as an abbreviation (non default), so that parsing the incorrect symbol will work until a major version. Can we mark abbreviation strings as obsolete @angularsen?

I don't think so, we will just have to mark it as a breaking change in #982 and document it, I guess.

[angularsen]

@Brunni If you can add the correct one first in the array, and keep the old one as second item in the array, then parsing won't break, while ToString() will prefer the first one. Then we can merge this.

[codecov]

Codecov Report

Merging #1125 (ccb54d8) into master (77b1ef2) will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master   #1125   +/-   ##
======================================
  Coverage      85%     85%           
======================================
  Files         322     322           
  Lines       52322   52322           
======================================
  Hits        44894   44894           
  Misses       7428    7428           

Impacted Files	Coverage Δ
UnitsNet/GeneratedCode/Quantities/Mass.g.cs	91% <100%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[angularsen]

Thank you! Nuget should be out in the next hour.

Release UnitsNet/4.145.0 · angularsen/UnitsNet