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

[release/8.0-rc1] Remove unnecessary TcpListener initialization #4325

Merged
merged 1 commit into from
Aug 28, 2023

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Aug 21, 2023

Backport of #4315 to release/8.0-rc1

/cc @joperezr @sebastienros

Customer Impact

Testing

Risk

IMPORTANT: If this backport is for a servicing release, please verify that:

  • The PR target branch is release/X.0-staging, not release/X.0.

  • If the change touches code that ships in a NuGet package, you have added the necessary package authoring and gotten it explicitly reviewed.

Microsoft Reviewers: Open in CodeFlow

@joperezr joperezr enabled auto-merge (rebase) August 21, 2023 19:12
@RussKie
Copy link
Member

RussKie commented Aug 28, 2023

@joperezr are we still taking changes in RC1?

@sebastienros
Copy link
Member

We should, any sign-off required?

@joperezr
Copy link
Member

Yeah, it's fine to take as long as build is green. No sign offs required for now.

@joperezr joperezr merged commit 0f8f4bc into release/8.0-rc1 Aug 28, 2023
6 checks passed
@joperezr joperezr deleted the backport/pr-4315-to-release/8.0-rc1 branch August 28, 2023 23:39
@ghost ghost locked as resolved and limited conversation to collaborators Sep 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants