Skip to content

Conversation

daxianji007
Copy link
Contributor

@daxianji007 daxianji007 commented Apr 28, 2017

Fix Missing .NET API documentation: https://mseng.visualstudio.com/VSChina/_workitems?id=979781

  • dv_ng_codepages_mref
  • dv_ng_dataflow_mref
  • dv_ng_immutable_mref
  • dv_ng_simd_mref

Fixes #1993

@dnfclas
Copy link

dnfclas commented Apr 28, 2017

@daxianji007,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.

It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

Copy link
Contributor

@mairaw mairaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some metadata information seems to be removed and you need to sign the CLA please. Thanks!

@@ -1,41 +1,23 @@
---
uid: System.Text.DecoderFallback
author: "rpetrusha"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why are these metadata values being removed?

@dnfclas
Copy link

dnfclas commented May 2, 2017

@daxianji007, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.

Thanks, .NET Foundation Pull Request Bot

@mairaw
Copy link
Contributor

mairaw commented May 2, 2017

Thanks for signing the CLA @daxianji007. In addition to my previous question, we also have build errors. Is that something you can take care of?

@daxianji007
Copy link
Contributor Author

@mairaw, I have updated the pull request:

  1. re-migrate the meta.md
  2. remove the empty token
    This should resolve the existing problems.

@@ -1,6 +1,6 @@
<Namespace Name="System.Text">
<Docs>
<summary>The <see cref="N:System.Text" /> namespace contains classes that represent ASCII and Unicode character encodings; abstract base classes for converting blocks of characters to and from blocks of bytes; and a helper class that manipulates and formats <see cref="T:System.String" /> objects without creating intermediate instances of <see cref="T:System.String" />.</summary>
<summary>The <see cref="N:System.Text" /> namespace in the System.Text.Encoding.CodePages.dll assembly contains a single class, <see cref="T:System.Text.CodePagesEncodingProvider" />, that extends the <see cref="T:System.Text.EncodingProvider" /> class to make code page encodings available to apps that target the Universal Windows Platform.</summary>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rpetrusha now that we'll only have a single namespace page for this, do you want to merge their content or leave the original one?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mairaw, Here I think that we want to leave the original one.

@@ -1,6 +1,6 @@
<Namespace Name="System.Numerics">
<Docs>
<summary>The <see cref="N:System.Numerics" /> namespace contains numeric types that complement the numeric primitives, such as <see cref="T:System.Byte" />, <see cref="T:System.Double" />, and <see cref="T:System.Int32" />, that are defined by the .NET Framework.</summary>
<summary>Provides a set of SIMD-enabled vector types.</summary>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we leave the original here @rpetrusha?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mairaw, Yes, the original one. Except for one type not in the .NET Framework, the .NET Framework SIMD documentation is identical to the .NET Core documentation.

@daxianji007
Copy link
Contributor Author

@mairaw , I fixed a bug that cause these missing Vector contents and had a re-migration.

@mairaw mairaw merged commit dde8b22 into dotnet:master May 18, 2017
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.

5 participants