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

Update azservicebus to latest go-amqp #20386

Merged
merged 1 commit into from
Mar 29, 2023

Conversation

jhendrixMSFT
Copy link
Member

  • The purpose of this PR is explained in this or a referenced issue.
  • The PR does not update generated files.
  • Tests are included and/or updated for code changes.
  • Updates to CHANGELOG.md are included.
  • MIT license headers are included in each file.

@jhendrixMSFT
Copy link
Member Author

/azp run go - azservicebus

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jhendrixMSFT
Copy link
Member Author

/azp run go - azservicebus

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jhendrixMSFT jhendrixMSFT changed the title go-amqp from flowcontrol branch azservicebus: go-amqp from flowcontrol branch Mar 10, 2023
@jhendrixMSFT
Copy link
Member Author

/azp run go - azservicebus

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jhendrixMSFT jhendrixMSFT changed the title azservicebus: go-amqp from flowcontrol branch Update azservicebus to latest go-amqp Mar 27, 2023
@jhendrixMSFT
Copy link
Member Author

/azp run go - azservicebus

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jhendrixMSFT jhendrixMSFT marked this pull request as ready for review March 29, 2023 18:23
@jhendrixMSFT
Copy link
Member Author

/azp run go - azservicebus

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@richardpark-msft richardpark-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. We have the LinkName() thing but I think we can merge this first on it's own and do the other one in just a moment.

@jhendrixMSFT jhendrixMSFT merged commit 880fad4 into Azure:main Mar 29, 2023
@jhendrixMSFT jhendrixMSFT deleted the azservicebus_go_amqp branch March 29, 2023 22:03
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.

2 participants