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

Comment parse #238

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

Comment parse #238

wants to merge 5 commits into from

Conversation

qRoC
Copy link

@qRoC qRoC commented Mar 16, 2023

Closes #224

  • whole comment parsing;
  • better Rust support;
  • handle comment as document.

Test builds: https://github.com/qRoC/ltex-ls/releases/tag/nightly

@qRoC qRoC changed the title Comment parse [DRAFT] Comment parse Mar 16, 2023
@qRoC qRoC changed the title [DRAFT] Comment parse Comment parse Mar 17, 2023
@ModProg
Copy link

ModProg commented Nov 28, 2023

I just tried this, and it seems to work. @valentjn what would need to happen for this to be merged?

@ModProg
Copy link

ModProg commented Dec 18, 2023

grafik

There seems to be some erroneous parsing.

Trigger seems to be linebreak in an inline codeblock (this might be something regular markdown does not support, i.e. rust doc comment specific. Minimal reproduction:

//!     - `
//!       ` Custom error message printed if an

@Freyskeyd
Copy link

Any updates regarding this PR? :)

@spitzerd
Copy link

spitzerd commented Dec 8, 2024

Your commits were merged into ltex-ls-plus:
ltex-plus@90358e3
ltex-plus@5e9e064
ltex-plus@e8d197a
ltex-plus@d48b875
ltex-plus@1b4d8d5

Thank you for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ignore nodes in code comments
4 participants