Skip to content

Draft: Enable MD001#28

Closed
Youssef1313 wants to merge 1 commit intodotnet:masterfrom
Youssef1313:md001
Closed

Draft: Enable MD001#28
Youssef1313 wants to merge 1 commit intodotnet:masterfrom
Youssef1313:md001

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.

2 participants