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

cli-plugins/manager: fix deprecation comment of Metadata.Experimental #4276

Conversation

thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented May 9, 2023

This field was marked deprecated in 977d3ae (#2774), which is part of v20.10 and up, but the comment was missing a newline before the deprecation message, which may be picked up by IDEs, but is not matching the correct format, so may not be picked up by linters.

This patch fixes the format, to make sure linters pick up that the field is deprecated.

- 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)

This field was marked deprecated in 977d3ae,
which is part of v20.10 and up, but the comment was missing a newline before
the deprecation message, which may be picked up by IDEs, but is not matching
the correct format, so may not be picked up by linters.

This patch fixes the format, to make sure linters pick up that the field is
deprecated.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah merged commit 185ac35 into docker:master May 11, 2023
@thaJeztah thaJeztah deleted the fix_cli_plugins_metadata_experimental_deprecation branch May 11, 2023 14:41
Sign up for free to 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.

2 participants