#291: move comments between imports above import list

[bethcutler]

Fix for #291

[JavierSegoviaCordoba]

Is this not a weird behavior? Imagine I am commenting on an import (for example //this is import is temporarily here and this dependency should be removed), the comment will be moved on top and the reference is lost.

[bethcutler]

I addressed this a bit in the #291 issue. The new behavior is better than the current behavior (these comments would just cause the formatter to crash, so effectively aren't "allowed" right now). Another option would be to pin the comment to the import just below the comment, but if the formatter removes that import because it is unnecessary, should the comment disappear? Now the comment may be "attached" to a completely unrelated import. I feel like this is the safest change we can make, and our users can easily adapt to it.

[nreid260]

Plus, if the comment is an import, that import will get sorted back into place once uncommented.

[cgrushko]

Let's first conclude the discussion in #291

[facebook-github-bot]

@strulovich has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.