Skip to content

api/types: move RequestPrivilegeFunc to api/types/registry#48119

Merged
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:deprecate_RequestPrivilegeFunc
Aug 30, 2024
Merged

api/types: move RequestPrivilegeFunc to api/types/registry#48119
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:deprecate_RequestPrivilegeFunc

Conversation

@thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Jul 2, 2024

api/types/registry: fix godoc, and add some doc-links

api/types: move RequestPrivilegeFunc to api/types/registry

Move the definition, but mostly keep it for documentation purposes,
to prevent having to import the registry package in all places.

- Description for the changelog

api/types: move RequestPrivilegeFunc to api/types/registry

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

@thaJeztah thaJeztah added this to the 28.0.0 milestone Jul 2, 2024
@thaJeztah thaJeztah self-assigned this Jul 2, 2024
@thaJeztah thaJeztah changed the title Deprecate request privilege func api/types: move RequestPrivilegeFunc to api/types/registry Jul 2, 2024
@thaJeztah thaJeztah force-pushed the deprecate_RequestPrivilegeFunc branch from 6bc375b to b567bc3 Compare August 13, 2024 12:34
@thaJeztah thaJeztah marked this pull request as ready for review August 30, 2024 12:04
@thaJeztah thaJeztah requested a review from vvoland August 30, 2024 12:04
@thaJeztah
Copy link
Member Author

CI cache is probably busted; let me rebase this one to get a fresh run.

Move the definition, but mostly keep it for documentation purposes,
to prevent having to import the registry package in all places.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the deprecate_RequestPrivilegeFunc branch from b567bc3 to bb899c6 Compare August 30, 2024 12:13
@thaJeztah thaJeztah merged commit 5ea96ca into moby:master Aug 30, 2024
@thaJeztah thaJeztah deleted the deprecate_RequestPrivilegeFunc branch August 30, 2024 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants