Skip to content

Conversation

Juliehzl
Copy link
Owner

@Juliehzl Juliehzl commented Aug 3, 2023

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.

Joanna-Yang-Art and others added 30 commits July 31, 2023 14:11
…etCore (Azure#37914)

* add EventSource to AspNetCore and cleanup

* update message
* standard metrics for messaging

* revert change
* update dotnet dev cert being shipped with the proxy to renew for another year
* update suppression file

---------

Co-authored-by: Scott Beddall <[email protected]>
* Marked empty/null actions as failed

* replaced tabs with spaces

* Updated spacing

* Replaced tab with spaces

* Updates based on feedback

* Updated Validation Logic

* WIP Added test cases for OneOrMoreRequiredAttribute

* Added test case for EnumberableItemsNotNull Attribute

* Feedback updates

* Updates based on PR feedback

* Sort 'using' lists and removed unapplicable data annotation

* Adding necessary updates to merge with azure main

* Removed xUnit reference now that we are using NUnit tests

* removed Update-Snippets.ps1 changes and will rebase

* Removing requirement for response object in request to not perform early validation

* Added null check and test for response before marking as failed

* Updated exception to validation exception and corrected test

* Changed back to argument null exception

* Added response validation exception type

* Ran script to add partial class for ResponseValidationException

* Update sdk/entra/Microsoft.Azure.WebJobs.Extensions.AuthenticationEvents/src/AuthenticationEventResource.resx

Adding a missing period

Co-authored-by: Jesse Squire <[email protected]>

* Added period

---------

Co-authored-by: Jesse Squire <[email protected]>
* merge Extensions classes

* update Options class

* changelog

* update public api

* remove nullables from public api

* changelog

* changelog

* export public api

* reset enum integer
* Update Generator Version 3.0.0-beta.20230801.2

* Update SDK codes

* Update SDK samples m*,n*,o*,p*,q*,r*

* Rename and update

---------

Co-authored-by: Wei Hu <[email protected]>
* Update Generator Version 3.0.0-beta.20230803.1

* Update SDK codes

* Update SDK codes

* Update SDK codes

* Update SDK codes
wsugarman and others added 10 commits August 3, 2023 08:05
* First pass

* Updated changelog and added comment

* Update sdk/extensions/Microsoft.Extensions.Azure/CHANGELOG.md

---------

Co-authored-by: Jesse Squire <[email protected]>
…ure#37935)

* Renamed API from API Arch Board Feedback

* Export API and Update Snippetes
* Marked empty/null actions as failed

* replaced tabs with spaces

* Updated spacing

* Replaced tab with spaces

* Updates based on feedback

* Updated Validation Logic

* WIP Added test cases for OneOrMoreRequiredAttribute

* Added test case for EnumberableItemsNotNull Attribute

* Feedback updates

* Updates based on PR feedback

* Sort 'using' lists and removed unapplicable data annotation

* Adding necessary updates to merge with azure main

* Removed xUnit reference now that we are using NUnit tests

* removed Update-Snippets.ps1 changes and will rebase

* Added null check and test for response before marking as failed

* Updated exception to validation exception and corrected test

* Changed back to argument null exception

* Added response validation exception type

* Update sdk/entra/Microsoft.Azure.WebJobs.Extensions.AuthenticationEvents/src/AuthenticationEventResource.resx

Adding a missing period

Co-authored-by: Jesse Squire <[email protected]>

* Added period

* Added not null check for source field in payload

* Changed source attribute to required

* Sorted usings

* Added test scenarios for AuthenticationEventMetadata

* Added big fix to changelog

* Refactored test case for consistency with existing tests and added links to changelog

* Added assertions for exception messages and removed links to bugs

* Ran scripts to update netstandard2.0 class

---------

Co-authored-by: Jesse Squire <[email protected]>
* bump the proxy version

* updating version proxy consumed by framework

---------

Co-authored-by: Scott Beddall <[email protected]>
* Add AMQP constructor to ServiceBusMessage

* Enable logging for test proxy integration

* Clean up

* readme

* add section

* revert default

* Add locking

* Add AspNetCore Info logging

* reduce delay to 20ms

* Put all proxy logging behind flag instead of allowing loglevel config

* Fix wrapping

* Split out CheckForErrors to make it sync
@Juliehzl Juliehzl force-pushed the user/zuh/containerapp0501 branch from 8c0ebd7 to 403a3e2 Compare August 4, 2023 05:17
azure-sdk and others added 15 commits August 4, 2023 10:19
Fixing a string that, while local only to the test instance, contains a secret that is causing security alerts.
* Fix MessageLockLost event test

* wrap in try/finally
…arameters to option bags (Azure#38006)

* Moved position parameter from StorageResourceSingle.WriteFromStreamAsync to options bag

* Made parameter `completeLength` from `StorageResourceSingle.CopyBlockFromUriAsync` mandatory.

* Removed extra dash in changelog
…re#38060)

* Prepare Azure.Core.Experimental release

* update

* for Krzysztof

* Export API

* fix

* fix
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.