exclude System.MonoTODo from API syntax

[mairaw]

I don't think we should be displaying that attribute in the syntax blocks for API docs.

Here's an example: https://docs.microsoft.com/en-us/dotnet/api/system.net.mail.mailaddress.-ctor?view=netframework-4.7#System_Net_Mail_MailAddress__ctor_System_String_System_String_System_Text_Encoding_

[mairaw]

@mhutch do you agree with this change?

[mairaw]

Doing a force publish on the branch to see if this change needs a full rebuild.

[mairaw]

https://review.docs.microsoft.com/en-us/dotnet/api/system.net.mail.mailaddress.-ctor?view=netframework-4.7&branch=mairaw-patch-3#System_Net_Mail_MailAddress__ctor_System_String_System_String_System_Text_Encoding_ still showing that attribute.

@dend who could help validate my changes to see what I'm doing wrong here?

[mhutch]

@mairaw makes sense to me! Perhaps they could go in some kinds of "platform availability notes" section, but they don't belong in the syntax block.

[BillWagner]

LGTM. when you are ready.

[mairaw]

It's not working. I'll open a new issue for the engineering team to investigate. For now, marking as WIP.

Thanks @mhutch. I'll open an issue for us to add that later on.

[mairaw]

@TianqiZhang anything else I should try?

[TianqiZhang]

@mairaw I merged your change to my branch tianzhDevelop, and this attribute is successfully filtered. So please merge this directly to master, it should work. We'll continue on improving incremental PR build in the future.

[mairaw]

Interesting. I also did full builds on mairaw-patch-3 and couldn't see the change on that branch too @TianqiZhang. But I'll merge and see the results. Thanks.