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

node: Remove SetInternalIPv4From Method #15873

Merged
merged 1 commit into from
Apr 27, 2021

Conversation

nathanjsweet
Copy link
Member

The SetInternalIPv4From method is no longer used
in any part of the codebase and is a red-herring when
trying to figure out places where SetInternalIPv4Router
might be called.

Signed-off-by: Nate Sweet nathanjsweet@pm.me

The `SetInternalIPv4From` method is no longer used
in any part of the code base and is a red-herring when
trying to figure out places where `SetInternalIPv4Router`
might be called.

Signed-off-by: Nate Sweet <nathanjsweet@pm.me>
@nathanjsweet nathanjsweet requested a review from a team as a code owner April 27, 2021 03:20
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Apr 27, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.10.0 Apr 27, 2021
@nathanjsweet nathanjsweet added kind/cleanup This includes no functional changes. release-note/misc This PR makes changes that have no direct user impact. labels Apr 27, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Apr 27, 2021
Copy link
Member

@gandro gandro left a comment

Choose a reason for hiding this comment

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

Reviewer note: Last use of this method was removed in 020c1ac

@aanm aanm merged commit c312584 into master Apr 27, 2021
1.10.0 automation moved this from In progress to Done Apr 27, 2021
@aanm aanm deleted the pr/nathanjsweet/remove-set-internal-ipv4-from branch April 27, 2021 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cleanup This includes no functional changes. release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants