Skip to content

First draft of Tables migration guide#21909

Merged
christothes merged 9 commits intoAzure:mainfrom
christothes:chriss/tablesMigrationGuide
Jun 22, 2021
Merged

First draft of Tables migration guide#21909
christothes merged 9 commits intoAzure:mainfrom
christothes:chriss/tablesMigrationGuide

Conversation

@christothes
Copy link
Copy Markdown
Member

No description provided.

@christothes christothes added the Client This issue is related to a non-management package label Jun 16, 2021
@christothes christothes self-assigned this Jun 16, 2021
@ghost ghost added the Tables label Jun 16, 2021
@christothes christothes requested a review from jsquire June 16, 2021 15:49
Copy link
Copy Markdown
Contributor

@seankane-msft seankane-msft left a comment

Choose a reason for hiding this comment

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

Couple of small things, LGTM

Comment thread sdk/tables/Azure.Data.Tables/MigrationGuide.md Outdated
Comment thread sdk/tables/Azure.Data.Tables/MigrationGuide.md Outdated
Comment thread sdk/tables/Azure.Data.Tables/MigrationGuide.md Outdated
Comment thread sdk/tables/Azure.Data.Tables/MigrationGuide.md Outdated
Comment thread sdk/tables/Azure.Data.Tables/MigrationGuide.md Outdated
Comment thread sdk/tables/Azure.Data.Tables/MigrationGuide.md Outdated
Comment thread sdk/tables/Azure.Data.Tables/MigrationGuide.md Outdated
Comment thread sdk/tables/Azure.Data.Tables/MigrationGuide.md Outdated
Comment thread sdk/tables/Azure.Data.Tables/MigrationGuide.md Outdated
Comment thread sdk/tables/Azure.Data.Tables/MigrationGuide.md Outdated
Comment thread sdk/tables/Azure.Data.Tables/MigrationGuide.md
Comment thread sdk/tables/Azure.Data.Tables/MigrationGuide.md
Comment thread sdk/tables/Azure.Data.Tables/MigrationGuide.md

For `Microsoft.Azure.Cosmos.cloudTable.TableEntity`:

```c#
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The nittest of nits: Should we consider normalizing to C# for the code fences? We're currently using c# for non-snippets and C# for snippets.

christothes and others added 2 commits June 17, 2021 07:59
Co-authored-by: Jesse Squire <jesse.squire@gmail.com>
Copy link
Copy Markdown
Member

@jsquire jsquire left a comment

Choose a reason for hiding this comment

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

:shipit:

Comment thread sdk/tables/Azure.Data.Tables/MigrationGuide.md Outdated
Comment thread sdk/tables/Azure.Data.Tables/MigrationGuide.md Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Client This issue is related to a non-management package Tables

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants