Skip to content
This repository has been archived by the owner on Feb 16, 2021. It is now read-only.

Linter based on rehype #1

Closed
denysdovhan opened this issue May 9, 2018 · 7 comments
Closed

Linter based on rehype #1

denysdovhan opened this issue May 9, 2018 · 7 comments
Labels
🧘 status/waiting This may go somewhere but needs more information

Comments

@denysdovhan
Copy link

remark-lint is awesome! Let's make the same thing for rehype!

@wooorm
Copy link
Member

wooorm commented May 11, 2018

Yes, that would be super! I started work on it a while ago, but was never able to finish it, as HTML has so many things to check for! 😬

@denysdovhan
Copy link
Author

@wooorm maybe there's a reason to start with reimplementing rules from existing linters, like htmlhint?

@wooorm
Copy link
Member

wooorm commented May 11, 2018

Definitely! I invited you to my private, not-really-working, initial work, maybe you’re into setting something up? I also have some newer code, with just some aria rules, that may be easier to convert to something, but it’s only on my computer 😬

Oh, and for inspiration:

@denysdovhan
Copy link
Author

I'll try to take a look at it!

@ChristianMurphy
Copy link
Member

On a related note, I created selective lint using a similar tech stack, rehype-parse, unified lint rule and unified message control.
The rule API is mainly focused on using CSS selectors as rules.

But the marker message control code could be used for a more generic linter. https://github.com/ChristianMurphy/selective/blob/master/%40selective/marker/marker.js
https://github.com/ChristianMurphy/selective/blob/master/%40selective/rehype/rehype.js

@wooorm
Copy link
Member

wooorm commented Jun 10, 2018

Not yet released but here’s a little bit! https://github.com/rehypejs/rehype/tree/master/packages/rehype-parse#optionsemitparseerrors

@wooorm wooorm added the 🧘 status/waiting This may go somewhere but needs more information label Aug 13, 2019
@ChristianMurphy
Copy link
Member

Thanks for starting the discussion @denysdovhan!
We're in the process unifying ideas in with discussions unifiedjs/collective#44
If you'd like to continue this thread, or start a new one https://github.com/rehypejs/rehype/discussions/categories/ideas will be the home for ideas going forward.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🧘 status/waiting This may go somewhere but needs more information
Development

No branches or pull requests

3 participants