Sync with remote master #3
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Changes based on .NET team and architect feedback:
- Use `Response<T>` in samples
- Use shorter names for the variables
- Use `using var` instead of a `using block` (easier to read)
- Add input variables that user will need to introduce for the code to work, like `formUri` or `formFilePath`.
- Avoid long lines so user doesn't have to scroll
- Replace block of code that look like this with real code:
```
using (FileStream stream = new FileStream(receiptPath, FileMode.Open))
{
RecognizedFormCollection receipts = await client.StartRecognizeReceipts(stream).WaitForCompletionAsync();
/*
*
*/
}
```
Fixes part of: #16661
Extending the build pipeline to allow project exclusions to be specified within the serivce directory. This is intended to enable splitting the pipeline into multiple definitions for service directories that contain both T1 and T2 packages, particularly in the case where those are maintained by different teams. Also included in this set of changes is a configuration set for the legacy Event Hubs packages.
Fixes: #17118 .NET 5 is available, by default, on all agents now. I'll keep the file so we don't have to re-discover all the places we need to install dotnet the next time.
…et/tree/master/samples (#17614) * Move CloudClipboard from https://github.com/Azure/azure-sdk-for-net-samples to https://github.com/Azure/azure-sdk-for-net/tree/master/samples * Move SimpleTracingPolicy class back to program * Keep consistency in app settings. Trying to resolve ci error. Co-authored-by: Frank Li (Wicresoft North America Ltd) <[email protected]>
The focus of these changes is to remove the legacy packages from the build pipeline definitions, as they're now covered under a dedicated set of pipeline definitions. This is intended to allow the T1 and T2 Event Hubs packages to be treated as separate units of work, as they will continue to evolve independently and are maintained by different teams.
The focus of these changes is to remove the Azure Function bindings projects from the legacy pipeline definitions.
…Live, Create, Get) (#17721)
* Add config for 1ES hosted Linux pool.
* Add AzureSasCredential * corner case. * api. * doc update. * less allocations. * call base last. * Revert "less allocations." This reverts commit 6375606. * validation feedback. * policy rename. * another attempt to reduce allocations. * Revert "another attempt to reduce allocations." This reverts commit 89cc867. * changelog.
* remove local swagger and re-record * short circuit ignored test setups
* Get Scope From Headers * Added Scope tests * Added a null check on Getting scope from headers * Renamed regex used in Getting scope from headers Co-authored-by: Clare Limo(INSIGHT MANAGEMENT CONSULTANTS) <[email protected]>
* Fix bug on create OCI Manifest * Added other manifests session records * Resolved PR comments * Updated the switch statement
…ception if GetPaths was called (#17698)
Co-authored-by: Chidozie Ononiwu <[email protected]>
* fixup LROs to conform to guidelines, add tests
Co-authored-by: jschrepp-MSFT <[email protected]>
PostConfigure ensures that the InitialOffsetOptions will be propagated to the EventProcessorOptions regardless of the ordering used when doing configuration.
- Add set of helpers to work with devops work items - Add script to create devops release package items - Update SemVer to support version type Co-authored-by: Wes Haggard <[email protected]>
Update AutoRest C# version to 3.0.0-beta.20210105.1
* Update latest folder with one index.html includes the redirect links * Update the redirect links and remove everything in latest * Address feedback * Have the entire copy for latest GA. Co-authored-by: Sima Zhu <[email protected]>
Update AutoRest C# version to 3.0.0-beta.20210115.1
* Used string.IsNullOrEmpty() in EnvironmentCredential * Update EnvironmentCredential.cs * Update CHANGELOG.md * Update CHANGELOG.md * Update CHANGELOG.md * adding tests for environment variable empty strings * reversing GetNonEmptyStringOrNull condition for readability Co-authored-by: Scott Schaab <[email protected]>
* Remove secure key release implementation Resolves #17901 * Update public APIs
Azure Spatial Anchors doesn't currently support West US 2, so the tests fail when run against that region. This change configures the test location to be a specific location (`eastus2`). This should resolve #17715.
Co-authored-by: Zhenyu Zhou <[email protected]>
* Add MachineLearningServices track 2 NET SDK
Helps follow up with the service team if any service issues occur during nightly live test runs.
* capture azurite debug log before failing permanently. * using... * pr feedback.
* regen the client
* Update Microsoft.Azure.Management.Compute.csproj back to 42 * Update AssemblyInfo.cs
* Added teams to sdk from API -Regenerated code using lastest API. -Updated strings in tests -Recoreded new test sessionrecords * Ran generate.ps1 * Update tests due to api changes * Version Update Updated assembly versions and release notes
- Code generator renamed to include prefix Azure/autorest.csharp#991
* Add PrepareRelease Script * Generalize Prepare-Release Script * Update Update-ChangeLog.ps1 * Update Package-Properties.ps1 * Update Collect-ChangeLog Script * Update Collect-ChangeLog.ps1 and Collect-Unreleased.ps1 * Update GeneralReleaseNotesParser.ps1 script * Update Prepare-Release Script to use recent DevOps Scripts * Move Get-LanguageName to eng\common\scripts\Helpers\DevOps-WorkItem-Helpers.ps1, improve error message, add logic to detect changes made by prepare-release script * Use LanguageDisplayName, move Get-CSVMetadata to Package-Properties.ps1 Co-authored-by: Chidozie Ononiwu <[email protected]>
- IntegrationRuntimesClientLiveTests.cs - LinkedServiceClientLiveTests.cs - NotebookClientLiveTests.cs - PipelineClientLiveTests.cs - PipelineRunClientLiveTests.cs - SparkJobDefinitionClientLiveTests.cs
Update AutoRest C# version to 3.0.0-beta.20210121.3
* Identity Derialization Requires Id * Add cloud environment to MicrosoftTeamsIdentifier * Update description Co-authored-by: Dominik <[email protected]>
…shOptions optional (#18149) * Communication - Make AsyncTokenRefresher in CommunicationTokenRefreshOptions optional * Update the tests to do not have custom logic in creation of token credential
* adding cloudservice generated sdk files and test * adding customizations to resolve runtime anbiguity * regenrating the sdk from rest repo * regenerating full SDK (including gallery and diskrp changes), fixing diskrp and gallery tests * regenrating SDK using rest repo
* fix endpoint env var name * add COMMUNICATION_ACCESS_KEY env var * change key1 to primaryKey
* Managed Identity for SMS * Adding readme env variables detail * Fix changelog * Adding live tests for managed identity SMS * Resolve merge conflicts * Remove extra connection string variable * Remove endpoint and parse from connection string * Updating tsts Co-authored-by: Minnie Liu <[email protected]>
* Add message batch factory * Export API
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.