Skip to content

Search SDK: Upgrading 4x-preview to vs17 and netcore#3208

Merged
brjohnstmsft merged 4 commits intoAzure:search-4x-preview-vs17from
brjohnstmsft:merge-to-4x-preview-vs17
May 16, 2017
Merged

Search SDK: Upgrading 4x-preview to vs17 and netcore#3208
brjohnstmsft merged 4 commits intoAzure:search-4x-preview-vs17from
brjohnstmsft:merge-to-4x-preview-vs17

Conversation

@brjohnstmsft
Copy link
Copy Markdown
Member

This re-applies the changes from the search-4x-preview branch to the new vs17Dev branch. See individual commits for details.

FYI @chaosrealm @Yahnoosh @mhko @bernitorres

Description


This checklist is used to make sure that common guidelines for a pull request are followed.

General Guidelines

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

SDK Generation Guidelines

  • If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
  • The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
  • The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

I tried rebasing the search-4x-preview branch to vs17Dev, but the results were
very messy, so instead this is just a squash-merge of all the changes. This
should differ from search-4x-preview only in the project files and some minor
test changes to account for behavioral changes in the latest version of
ClientRuntime.
@shahabhijeet
Copy link
Copy Markdown
Contributor

@azuresdkci test this please

2 similar comments
@shawns1
Copy link
Copy Markdown

shawns1 commented May 15, 2017

@azuresdkci test this please

@shahabhijeet
Copy link
Copy Markdown
Contributor

@azuresdkci test this please

@shahabhijeet
Copy link
Copy Markdown
Contributor

@azuresdkci test this please

1 similar comment
@shahabhijeet
Copy link
Copy Markdown
Contributor

@azuresdkci test this please

@brjohnstmsft brjohnstmsft requested review from shahabhijeet and removed request for seansaleh and shahabhijeet May 15, 2017 23:39
@brjohnstmsft brjohnstmsft merged commit c3500ca into Azure:search-4x-preview-vs17 May 16, 2017
@brjohnstmsft brjohnstmsft deleted the merge-to-4x-preview-vs17 branch May 16, 2017 00:06
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.

4 participants