Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Auto rest composite client #2117

Closed
wants to merge 1,552 commits into from
Closed

Auto rest composite client #2117

wants to merge 1,552 commits into from

Conversation

pankajsn
Copy link
Contributor

Integration account updates in the Logic App

Hovsep Mkrtchyan and others added 30 commits April 27, 2016 16:30
Revert "notificationHub changes using swagger file"
And include diagnostics.
…into AutoRest

# Conflicts:

#	src/ResourceManagement/DataLake.Analytics/DataLakeAnalytics.Tests/SessionRecords/DataLakeAnalytics.Tests.CatalogOperationTests/GetCatalogItemsTest.json

#	src/ResourceManagement/DataLake.Analytics/DataLakeAnalytics.Tests/SessionRecords/DataLakeAnalytics.Tests.CatalogOperationTests/SecretAndCredentialCRUDTest.json

#	src/ResourceManagement/DataLake.Analytics/Microsoft.Azure.Management.DataLake.Analytics/Generated/Models/USqlJobProperties.cs
Forgot to rev the version and added a missing property
Also re-recorded all session records.
Search SDK: Upgrading to the latest version of AutoRest
Missing enum values in ADLA
This is related to the following issue: Azure/autorest#382

Azure Search doesn't support renaming resources, so strictly speaking the
overloads that take an extra name parameter are not required, but it doesn't
hurt anything to have them there in case we support rename in the future. At
least by moving our preferred overloads of CreateOrUpdate to custom code, we
can stop fighting the code generator.
Search SDK: Allowing redundant CreateOrUpdate methods to be generated
Update project.json and assemblyInfo to fix file version problem with RedisCache
Initial commit of the DevTestLabs SDK.
NotificationHub generated from Swagger file
sichatro and others added 23 commits June 9, 2016 15:32
Fixed RedisCache project not opening in VS 2015.
Updated RunCredScan script to check arguments path and copy filters file from repo
…ion for net45 TPM

Referenced new ClientRuntimes

Fixed assembly version.
Changed reference to ClientRuntimes due to downgrade of Json.Net version for net45 TPM
…into AutoRest

Conflicts:
	src/ResourceManagement/RedisCache/Microsoft.Azure.Management.Redis/Properties/AssemblyInfo.cs
	src/ResourceManagement/RedisCache/Microsoft.Azure.Management.Redis/project.json
Downgrading the runtime version to allow current test patterns in mocks
…into AutoRest

# Conflicts:

#	src/ResourceManagement/DataLake.Analytics/DataLakeAnalytics.Tests/project.json

#	src/ResourceManagement/DataLake.Analytics/Microsoft.Azure.Management.DataLake.Analytics/Properties/AssemblyInfo.cs

#	src/ResourceManagement/DataLake.Store/Microsoft.Azure.Management.DataLake.Store/Properties/AssemblyInfo.cs
Removing location property from record set resource as not tracked. Changing session recordings for tests.
Schedule Patching operations added
Use latest resourcemanager and datalake.store
Updated RunCredScan script to display better messages
Update based on latest specs:
- Adds TableTypes functionality
- Adds DistributionInfo to table objects

Test updates:
- Remove unsupported test
- Update test logic due to bug in the front end requiring jobId in
payload.
@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@azurecla
Copy link

Hi @pankajsn, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!


It looks like you're working at Microsoft (pankajsn). If you're full-time, we DON'T require a contribution license agreement.



If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@markcowl
Copy link
Member

@pankajsn looks like you have targeted the wrong branch. I think you wanted to target the AutoRest branch

@pankajsn
Copy link
Contributor Author

I targeted the AutoRest branch earlier but I got confused with the documentation.
https://github.com/Azure/adx-documentation-pr/blob/master/engineering/autorest-to-powershell.md#standard-process
#10.ii

I will send the new pull request.

Regards,
~Pankaj

From: Mark Cowlishaw [mailto:[email protected]]
Sent: Monday, June 20, 2016 3:26 PM
To: Azure/azure-sdk-for-net [email protected]
Cc: Pankaj Singh Negi [email protected]; Mention [email protected]
Subject: Re: [Azure/azure-sdk-for-net] Auto rest composite client (#2117)

@pankajsnhttps://github.com/pankajsn looks like you have targeted the wrong branch. I think you wanted to target the AutoRest branch


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com//pull/2117#issuecomment-227288691, or mute the threadhttps://github.com/notifications/unsubscribe/APZfc1krusi5g_XR2q7zwzS3G5UL8cuTks5qNxNvgaJpZM4I6JoC.

@pankajsn
Copy link
Contributor Author

Removing this pull request & creating a new one targeting AutoRest branch.

@pankajsn pankajsn closed this Jun 20, 2016
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.