analyzer

Hey @twostraws ! 👋🏼

Based on the screenshot here: https://twitter.com/twostraws/status/1354136797496881154/photo/1

I was wondering why imports are word-wrapped instead of formatted like everything else?

I think a format like this would be easier to read:

Imports
    UIKit: 8
    Foundation: 63
    AppKit: 17

Structure:
    UIKit View Controllers: 17
    UIKit Views: 23

Is your feature request related to a problem? Please describe.
Add a rule that fails when unstable package is used (e.g foo@1.0.0-alpha)

Is your feature request related to a problem? Please describe.

No, I wouldn't say it's a "problem", but: At least for me, when VSTHRD111 happens, and I "Alt-Enter" to get to the suggested code fixes, the first code fix listed is always to use ConfigureAwait(true). Since it is the first listed, it is the default choice when you hit "Alt-Enter" then "Enter".

In my experience, that's a

As per the respective RFCs, IGMP uses MRD in units of 1/10s while MLD uses units of ms.

Currently the Ostinato GUI shows units as 1/10s for both IGMP and MLD.

Please describe what the rule should do:
If 0 values are mostly passed into EdgeInsets.fromLTRB then it makes sense to use EdgeInsets.only for only the values that are to be used.
Also if there are symmetric values passed into EdgeInsets.fromLTRB or EdgeInsets.only then it makes sense to convert this into the EdgeInsets.symmetric equivalent.
In general I also find EdgeInsets.fromLTRB less