Skip to content

cli/context/docker: rename receiver for Endpoint#4257

Merged
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:context_rename_receiver
May 11, 2023
Merged

cli/context/docker: rename receiver for Endpoint#4257
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:context_rename_receiver

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

Code in methods of this type also used the Client, and having this receiver named "c" made it easy to confuse it for referring to Client ("c").

- What I did

- How I did it

- How to verify it

- Description for the changelog

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

Code in methods of this type also used the Client, and having this receiver
named "c" made it easy to confuse it for referring to Client ("c").

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added status/2-code-review area/context kind/refactor PR's that refactor, or clean-up code labels May 4, 2023
@thaJeztah thaJeztah merged commit 935df5a into docker:master May 11, 2023
@thaJeztah thaJeztah deleted the context_rename_receiver branch May 11, 2023 22:06
@thaJeztah thaJeztah added this to the 25.0.0 milestone May 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/context kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants