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

Merge main to feature/amqp starit up merge #4665

Merged
merged 13 commits into from
May 30, 2023
Merged

Conversation

gearama
Copy link
Member

@gearama gearama commented May 30, 2023

Pull Request Checklist

Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:

See the detailed list in the contributing guide.

  • C++ Guidelines
  • Doxygen docs
  • Unit tests
  • No unwanted commits/changes
  • Descriptive title/description
    • PR is single purpose
    • Related issue listed
  • Comments in source
  • No typos
  • Update changelog
  • Not work-in-progress
  • External references or docs updated
  • Self review of PR done
  • Any breaking changes?

benbp and others added 13 commits May 16, 2023 16:32
Co-authored-by: Shawn Fang <[email protected]>
* Adding initial commit of TypeSpecE2E common script

* add TODO comment

* Updated per feedback

---------

Co-authored-by: raychen <[email protected]>
… operations. (#4627)

* add test helper

* constructible from Args

* restucture ClassTraits to ensure portability of static constexpr

* fix Etags tests and format

* clang format
* Add DefaultAzureCredential auth flow diagram

* Update cspell.json

---------

Co-authored-by: Anton Kolesnyk <[email protected]>
Co-authored-by: James Suplizio <[email protected]>
#4640)

* Prep for readonly connection

* Added options for Connection and Session to enable read-only configuration

* Read-only connection and session objects; use message source and target for message sender and receiver

* Enabled read-only session; added tests for incorrect AMQPS port

* Added token reader sample; flushed out eventhubs properties sample, added some timeouts to send async test

* Moved AMQP samples to internal

* Moved samples to use GetEnvHelper

---------

Co-authored-by: Anton Kolesnyk <[email protected]>
* Standardized header ordering for C++ repo

* Reordered categories to move private headers above public headers; added space between categories to enable future flexibility
@github-actions github-actions bot added Azure.Core Azure.Identity EngSys This issue is impacting the engineering system. KeyVault Storage Storage Service (Queues, Blobs, Files) labels May 30, 2023
@gearama gearama changed the title Merge main amqp Merge main to feature/amqp starit up merge May 30, 2023
@gearama gearama merged commit 892bac8 into feature/amqp May 30, 2023
@gearama gearama deleted the merge_main_amqp branch May 30, 2023 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Attestation Azure.Core Azure.Identity EngSys This issue is impacting the engineering system. KeyVault Storage Storage Service (Queues, Blobs, Files)
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

7 participants