[wip] support for all recommended xml tags

[ademar]

The purpose of this PR is to bring support for all recommended tags (ref #295)

https://msdn.microsoft.com/en-us/library/vstudio/5ast78ax(v=vs.100).aspx

TODO: unit tests

[tpetricek]

Very cool, it will be great to have this!

[matthid]

Yeah I think this is very useful, thanks! Can you rebase your changes against the latest changes (ideally against the fix_extension_methods branch)?

[ademar]

Guys, I have rebased this PR against master.

I have disabled this test 4f4c3c2

In our use case, markDownComments = false, we want to render all comments regardless of whether or not they are enclosed by XML tags.

It would be nice if this gets merged, I could use it over here SuaveIO/suave#239

[matthid]

There are multiple reasons why I think we should support special commands, even when markDownComments = false:

• It should be as simple as for the markdown case to exclude some comments from the website (while still having them in the IDE).
• Problem is that sometimes FSharp.Compiler.Service throws errors on some language constructs, and [omit] is an easy workaround for such cases.

So it was definitely "by-design" that I called https://github.com/tpetricek/FSharp.Formatting/pull/331/files#diff-e2212151e1f9b6e7e085820d2a6b71caL652 (findCommands) in both cases.

[matthid]

@tpetricek What do you think?

The discussion is about using

/// [omit]

in case of markDownComments = false, where we ignore other markdown constructs.

[ademar]

@matthid Sorry, I now realize I did not understood correctly the purpose of findCommand and [omit]. Let me correct that.