Unity-Technologies / com.unity.mobile.notifications

Mobile Notifications Package
https://docs.unity3d.com/Packages/com.unity.mobile.notifications@2.1/manual/index.html
Other
132 stars 42 forks source link

Formatting update #255

Closed aurimasc closed 1 year ago

aurimasc commented 1 year ago

Two changes here:

Main change is the later. It happened more than once when formatter update actually did minor changes to format and perfectly formatted code became incorrectly formatted. This shouldn't happen, repository should use a specific fixed formatter and consciously update it.

aurimasc commented 1 year ago

Not sure I agree, because that means your package might not follow latest official formatter rules.

That's the problem: the formatter defines what the rules are, not merely enforce them. And new versions of formatter come out with not much warning. A well formatted code should not become badly formatted without code changes, that's why I'm pinning the version.