Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

multiline issues only display the first line #180

Open
drahnr opened this issue Jun 8, 2021 · 1 comment
Open

multiline issues only display the first line #180

drahnr opened this issue Jun 8, 2021 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@drahnr
Copy link
Owner

drahnr commented Jun 8, 2021

Describe the bug

To Reproduce

Steps to reproduce the behaviour:

  1. A file containing non\n fungible
  2. Run cargo spellcheck ...
  3. ...
error: spellcheck(NlpRules)
   --> /media/supersonic1t/projects/parity/polkadot/xcm/src/v0/multi_asset.rs:82
    |
 82 |  consensus system relative to the context interpreting. Use of a `MultiLocation` ensures that similar but non
    |                                                                                                           ^^^
    | - non-fungible
    |
    |   It appears that a hyphen is missing.

Expected behavior

Show two lines with both segments underlined.

Please complete the following information:

  • System: Fedora
  • Obtained: cargo
  • Version: 0.8.5

Additional context

@drahnr drahnr added the bug Something isn't working label Jun 8, 2021
@drahnr drahnr self-assigned this Jun 8, 2021
@drahnr
Copy link
Owner Author

drahnr commented Aug 19, 2021

Even more trivial example: The * splits the issue into two segments, but the underline cannot handle that.

error: spellcheck(NlpRules)
    --> /media/supersonic1t/projects/parity/polkadot-bernhard-malus-fx/xcm/src/v1/order.rs:108
     |
 108 |  - `effects`: The orders to execute on the assets once arrived *on the destination location*.
     |                                                        ^^^^^^^^
     | - arrived at
     |
     |   The usual preposition after "arrived" is "at", not "on". Did you mean arrived at?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant