Skip to content

Conversation

@v-dharmarajv
Copy link
Member

Contributing to the Azure SDK

Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.

For specific information about pull request etiquette and best practices, see this section.

hallipr and others added 30 commits October 30, 2024 13:35
- Use eng/common emitter archetype template
- Move build steps from yml to Build-Emitter.ps1
- Move test steps from yml to Test-Emitter.ps1
- Remove Generate.psm1 in favor of functions in Generate-Code.ps1
- Move eng/packages/*/eng/scripts to /eng/scripts/typespec
* [CODEOWNERS] Add CodeGen/Azure Plugin reviewers

The focus of these changes is to add the PR labels and reviewers for the code generation infrastructure and Azure generation plugins.
* Address a few AOT warnings in Azure.Core

* Fix apicompat
* Annotate Azure.Storage.Blobs for trim/AOT compat

* Fix apicompat

* Fix trim warning message
* Annotate Azure.Security.KeyVault.Secrets for trim/AOT compat

* Remove ExpectedWarningsFilepath
* package-properties are now populated with matrix configurations from their ci.yml if present
* create new code path for generate-job-matrix.yml which combines Create-JobMatrix and the "distribute-packages-to-matrix" action to generate dynamic matrices for PRs

---------

Co-authored-by: Scott Beddall <[email protected]>
* add mock test autorest file

* fix added test

* tests and assets changes

* add new delete test recordings

* added test recording
* identify and resolve missing function GenerateMatrixForConfig 
* identify and resolve miss-used null-coalesce operator

---------

Co-authored-by: Scott Beddall <[email protected]>
* update SDK for 2024-09-01-preview version

* update changelog

* add final Swagger commit id

* update changelog

* run eng\scripts\Export-API.ps1 CosmosDB

* update changelog

* update test recordings

* update tests to check for Mode != RecordedTestMode.Playback

* re-record tests

* re-record tests

* Review comments

* Modified enum name

* Regenerated code

* re-record test

---------

Co-authored-by: Praful Johari <[email protected]>
Co-authored-by: pjohari-ms <[email protected]>
* Update Logging

* Update eng/common/scripts/logging.ps1

---------

Co-authored-by: Chidozie Ononiwu <[email protected]>
Co-authored-by: Wes Haggard <[email protected]>
…parallel could cause a Dictionary Collision in the transfers stored (#46919)

* WIP

* WIP -test

* Fixed bug where adding multiple transfers in parallel could cause a collision

* Merge main test continued

* Fix tests
)

* skip last line of breaking change log

* add log

* retrieve breaking change from log
@v-dharmarajv v-dharmarajv merged commit 24beab9 into callautomation/beta3 Dec 10, 2024
140 checks passed
@v-dharmarajv v-dharmarajv deleted the users/v-vdharmaraj/beta3_merge_fron_main branch December 10, 2024 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.