Skip to content

remove some remnants from CLI "experimental" config option#6235

Merged
thaJeztah merged 2 commits intodocker:masterfrom
thaJeztah:remove_cli_experimental_remnants
Aug 6, 2025
Merged

remove some remnants from CLI "experimental" config option#6235
thaJeztah merged 2 commits intodocker:masterfrom
thaJeztah:remove_cli_experimental_remnants

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah commented Aug 3, 2025

Experimental is always enabled (977d3ae), and the Experimental field in plugin metadata was deprecated in 977d3ae and removed in commit 6a50c4f.

The second commit reformats a test-table for readability.

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

@thaJeztah thaJeztah added this to the 29.0.0 milestone Aug 3, 2025
@thaJeztah thaJeztah force-pushed the remove_cli_experimental_remnants branch from 58a23b3 to 6853c8d Compare August 3, 2025 16:03
@thaJeztah thaJeztah marked this pull request as draft August 4, 2025 06:27
Experimental is always enabled (977d3ae),
and the `Experimental` field in plugin metadata was deprecated in
977d3ae and removed in commit
6a50c4f.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Slightly more verbose, but makes it easier to see properties
of each test.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the remove_cli_experimental_remnants branch from 6853c8d to 057f312 Compare August 4, 2025 06:51
@thaJeztah thaJeztah marked this pull request as ready for review August 4, 2025 06:51
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
e2e/internal/fixtures/fixtures.go 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@thaJeztah thaJeztah requested review from Benehiko and vvoland August 5, 2025 16:50
@thaJeztah thaJeztah merged commit 7dd9c20 into docker:master Aug 6, 2025
105 of 106 checks passed
@thaJeztah thaJeztah deleted the remove_cli_experimental_remnants branch August 6, 2025 07:50
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.

3 participants