-
Notifications
You must be signed in to change notification settings - Fork 95
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
Markdown linting rules #432
Comments
I’m open to a PR addressing this 😃 I’m not sure it’s a fit for every project since "bold" markup should not depends on translations. The URL one ensure that the same information is on every translations but I’m not sure that the same number of "bold" or "italic" markup is a good check. |
Maybe it would make sense to allow to configure which checks do you want to apply on the dashboard? From what I've seen, the call that is made on the dashboard requests for a lint specifying the complete list of checks. I'm thinking on allowing to configure a project wide setting that defines which ones are active or not. |
We are using for our strings markdown syntax to be able to define bold and links from the string itself.
Will it make sense for this project, if we open a PR adding two new checks (one for bold, and one for links) that compare the count of bold matches between the master value and the value?
Similar to the current check for
url_count
(lib/lint/checks/url_count.ex).The text was updated successfully, but these errors were encountered: