Skip to content

Commit

Permalink
test: update snapshot with fixed annotation but carets include whites…
Browse files Browse the repository at this point in the history
…pace

I separated out this snapshot update since the string of `^` including
whitespace looked a little odd. I investigated this one specifically,
and indeed, our span in this case is telling `annotate-snippets` to
point at the whitespace. So this is `annotate-snippets` doing what it's
told with a mildly sub-optimal span.

For clarity, the before rendering is:

    skip.py:34:1: I001 [*] Import block is un-sorted or un-formatted
       |
    32 |       import sys; import os  # isort:skip
    33 |       import sys; import os  # isort:skip  # isort:skip
    34 | /     import sys; import os
       |
       = help: Organize imports

And now after is:

    skip.py:34:1: I001 [*] Import block is un-sorted or un-formatted
       |
    32 |     import sys; import os  # isort:skip
    33 |     import sys; import os  # isort:skip  # isort:skip
    34 |     import sys; import os
       | ^^^^^^^^^^^^^^^^^^^^^^^^^ I001
       |
       = help: Organize imports

This is a clear bug fix since it adds in the `I001` annotation, even
though the carets look a little funny by including the whitespace
preceding `import sys; import os`.
  • Loading branch information
BurntSushi committed Jan 8, 2025
1 parent 155a7b3 commit ae2e504
Showing 1 changed file with 10 additions and 10 deletions.
Original file line number Diff line number Diff line change
@@ -1,16 +1,15 @@
---
source: crates/ruff_linter/src/rules/isort/mod.rs
snapshot_kind: text
---
skip.py:20:1: I001 [*] Import block is un-sorted or un-formatted
|
18 | import sys
19 | import os # isort: skip
20 | / import collections
21 | | import abc
22 | |
| |_^ I001
23 |
| |_______________^ I001
22 |
23 |
24 | def f():
|
= help: Organize imports
Expand All @@ -32,9 +31,9 @@ skip.py:27:1: I001 [*] Import block is un-sorted or un-formatted
26 | import os # isort:skip
27 | / import collections
28 | | import abc
29 | |
| |_^ I001
30 |
| |_______________^ I001
29 |
30 |
31 | def f():
|
= help: Organize imports
Expand All @@ -52,9 +51,10 @@ skip.py:27:1: I001 [*] Import block is un-sorted or un-formatted

skip.py:34:1: I001 [*] Import block is un-sorted or un-formatted
|
32 | import sys; import os # isort:skip
33 | import sys; import os # isort:skip # isort:skip
34 | / import sys; import os
32 | import sys; import os # isort:skip
33 | import sys; import os # isort:skip # isort:skip
34 | import sys; import os
| ^^^^^^^^^^^^^^^^^^^^^^^^^ I001
|
= help: Organize imports

Expand Down

0 comments on commit ae2e504

Please sign in to comment.