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

fix(markdown): increase regex pattern specificity #903

Closed
wants to merge 1 commit into from
Closed

fix(markdown): increase regex pattern specificity #903

wants to merge 1 commit into from

Conversation

lostl1ght
Copy link

Fixes #902

@lostl1ght lostl1ght changed the title fix(markdown): inscrease regex pattern specificity fix(markdown): increase regex pattern specificity Jul 15, 2024
@folke
Copy link
Owner

folke commented Jul 15, 2024

Sorry, but that's just not a valid code-block language.
I'm not going to add bug fixes for LSPs in Noice....

@folke folke closed this Jul 15, 2024
@lostl1ght
Copy link
Author

Well, default hover is not crashing with the same code block:
image

IMO 'noice' should match that behavior.

@folke
Copy link
Owner

folke commented Jul 15, 2024

I'm fine with gracefully handling these types of errors, but not to fix bugs in a certain LSP.
So feel free to update the PR to do just that

@folke folke reopened this Jul 15, 2024
@lostl1ght
Copy link
Author

What do you mean by handling this error gracefully? Should noice send an error instead of trying to render a hover message if it has invalid markdown or do something else?

Also, apparently it is not a rust-analyzer error but an error in that library's documentation 😅

@folke folke closed this in 3b5ea55 Jul 15, 2024
folke pushed a commit that referenced this pull request Jul 17, 2024
🤖 I have created a release *beep* *boop*
---


## [4.4.3](v4.4.2...v4.4.3)
(2024-07-16)


### Bug Fixes

* **mini:** no need to show scrollbar
([c661f95](c661f95))
* **syntax:** prevent invalid group names. Closes
[#903](#903)
([3b5ea55](3b5ea55))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

bug: Invalid character in group name
2 participants