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

Initial conversion of HTML to markdown #1183

Merged
merged 9 commits into from
Aug 25, 2015

Conversation

sharwell
Copy link
Member

The conversion was performed using the following PowerShell script.

Function TransformStyleCop($file)
{
  $xslt = New-Object System.Xml.Xsl.XslCompiledTransform
  $xslt.Load($pwd.Path + '\TranslateToMarkdown.xslt')
  $xslt.Transform($pwd.Path + '\' + $file, $pwd.Path + '\' + [System.IO.Path]::ChangeExtension($file, '.md'))
}

Get-ChildItem *.html | ? { TransformStyleCop($_.Name) }

Each of the Markdown files still needs to be verified for accuracy against the original documentation for the diagnostic. After the verification is complete, we can work to document any known differences between StyleCopAnalyzers and StyleCop "classic".

Verification checklist

  • Spacing rules
    • SA1000
    • SA1001
    • SA1002
    • SA1003
    • SA1004
    • SA1005
    • SA1006
    • SA1007
    • SA1008
    • SA1009
    • SA1010
    • SA1011
    • SA1012
    • SA1013
    • SA1014
    • SA1015
    • SA1016
    • SA1017
    • SA1018
    • SA1019
    • SA1020
    • SA1021
    • SA1022
    • SA1023
    • SA1024
    • SA1025
    • SA1026
    • SA1027
    • SA1028
  • Readability rules
    • SA1100
    • SA1101
    • SA1102
    • SA1103
    • SA1104
    • SA1105
    • SA1106
    • SA1107
    • SA1108
    • SA1109
    • SA1110
    • SA1111
    • SA1112
    • SA1113
    • SA1114
    • SA1115
    • SA1116
    • SA1117
    • SA1118
    • SA1120
    • SA1121
    • SA1122
    • SA1123
    • SA1124
    • SA1125
    • SA1126
  • Ordering rules
    • SA1200
    • SA1201
    • SA1202
    • SA1203
    • SA1204
    • SA1205
    • SA1206
    • SA1207
    • SA1208
    • SA1209
    • SA1210
    • SA1211
    • SA1212
    • SA1213
    • SA1214
    • SA1215
    • SA1216
    • SA1217
  • Naming rules
    • SA1300
    • SA1301
    • SA1302
    • SA1303
    • SA1304
    • SA1305
    • SA1306
    • SA1307
    • SA1308
    • SA1309
    • SA1310
    • SA1311
  • Maintainability rules
    • SA1119
    • SA1400
    • SA1401
    • SA1402
    • SA1403
    • SA1404
    • SA1405
    • SA1406
    • SA1407
    • SA1408
    • SA1409
    • SA1410
    • SA1411
    • SA1412
  • Layout rules
    • SA1500
    • SA1501
    • SA1502
    • SA1503
    • SA1504
    • SA1505
    • SA1506
    • SA1507
    • SA1508
    • SA1509
    • SA1510
    • SA1511
    • SA1512
    • SA1513
    • SA1514
    • SA1515
    • SA1516
    • SA1517
    • SA1518
    • SA1519
    • SA1520
  • Documentation rules
    • SA1600
    • SA1601
    • SA1602
    • SA1603
    • SA1604
    • SA1605
    • SA1606
    • SA1607
    • SA1608
    • SA1609
    • SA1610
    • SA1611
    • SA1612
    • SA1613
    • SA1614
    • SA1615
    • SA1616
    • SA1617
    • SA1618
    • SA1619
    • SA1620
    • SA1621
    • SA1622
    • SA1623
    • SA1624
    • SA1625
    • SA1626
    • SA1627
    • SA1628
    • SA1629
    • SA1630
    • SA1631
    • SA1632
    • SA1633
    • SA1634
    • SA1635
    • SA1636
    • SA1637
    • SA1638
    • SA1639
    • SA1640
    • SA1641
    • SA1642
    • SA1643
    • SA1644
    • SA1645
    • SA1646
    • SA1647
    • SA1648
    • SA1649
    • SA1650
    • SA1651
    • SA1652
  • Alternative rules
    • SX1309
    • SX1309S

@sharwell
Copy link
Member Author

📝 This pull request is a straight conversion of the original HTML content to Markdown. I'm planning to leave all editing, including updates for disabled rules and intentional deviations from StyleCop "classic", for later updates.

@pdelvo @vweijsters Any reason not to merge this?

@vweijsters
Copy link
Contributor

👍 Looks good to me

@pdelvo
Copy link
Member

pdelvo commented Aug 24, 2015

It might be a good idea to write a unit test that verifies that all help links pinkt to an url on github to make sure we dont forget any diagnostics.

@otac0n
Copy link
Contributor

otac0n commented Aug 24, 2015

@pdelvo 👍

@sharwell
Copy link
Member Author

@pdelvo @otac0n Added tests.

sharwell added a commit that referenced this pull request Aug 25, 2015
Initial conversion of HTML to markdown
@sharwell sharwell merged commit 5686d67 into DotNetAnalyzers:master Aug 25, 2015
@sharwell sharwell deleted the add-docs branch August 25, 2015 03:12
@otac0n
Copy link
Contributor

otac0n commented Aug 25, 2015

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants