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

Draft: Enable MD001 #28

Closed
wants to merge 1 commit into from
Closed

Conversation

Youssef1313
Copy link
Member

This is to see how big the violations of this rule.

The documentation of the rule mentions that violations affect accessibility scenarios. Given that this repo is much smaller than dotnet/docs, the violations could be manageable to fix. If they are too many, I'll close.

@Youssef1313
Copy link
Member Author

There are 261 violation in a total of 230 file. I'm not sure if you consider this number large or not. Let me know if you want to fix these or not.

and in any case, if there are more articles to be moved from dotnet/docs, I don't think it's a good idea to enable the rule.

@adegeo
Copy link
Contributor

adegeo commented Sep 3, 2020

Good idea.

How about we export the results into a new issue. This way we can break the fixes up into manageable (and reviewable) parts and let the community have opportunity to contribute to the work.

@Youssef1313
Copy link
Member Author

@adegeo, Great. I've opened an issue to track that work. How much violations do you think is reasonable to be fixed per PR?

@adegeo
Copy link
Contributor

adegeo commented Sep 4, 2020

20

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.

None yet

2 participants