Skip to content

cli/command: remove deprecated ElectAuthServer() - #4110

Merged
thaJeztah merged 1 commit into
docker:masterfrom
thaJeztah:remove_ElectAuthServer
Mar 22, 2023
Merged

cli/command: remove deprecated ElectAuthServer()#4110
thaJeztah merged 1 commit into
docker:masterfrom
thaJeztah:remove_ElectAuthServer

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

This function was deprecated in b4ca1c7 (#2819), which is part of the v23.0 release, and is no longer used, so we can remove it.

- Description for the changelog

- cli/command: remove deprecated ElectAuthServer()

- A picture of a cute animal (not mandatory but encouraged)

This function was deprecated in b4ca1c7,
which is part of the v23.0 release, and is no longer used, so we can remove it.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #4110 (a3d56e7) into master (bfe87fd) will increase coverage by 0.00%.
The diff coverage is n/a.

Additional details and impacted files
@@ Coverage Diff @@## master #4110 +/- ##
=======================================
Coverage 59.14% 59.15% =======================================
Files 287 287 Lines 24741 24739 -2 =======================================
Hits 14634 14634 + Misses 9221 9219 -2 
Partials 886 886 

@laurazardlaurazard left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@thaJeztah

Copy link
Copy Markdown
MemberAuthor

Thx! Let me bring this one in 👍

@thaJeztah
thaJeztah merged commit 1448258 into docker:masterMar 22, 2023
@thaJeztah
thaJeztah deleted the remove_ElectAuthServer branch March 22, 2023 18:30
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@thaJeztah@codecov-commenter@laurazard