Skip to content

Fix buildkit flags not being hidden without buildkit enabled - #2735

Merged
silvin-lubecki merged 1 commit into
docker:masterfrom
thaJeztah:fix_flag_hiding
Oct 29, 2020
Merged

Fix buildkit flags not being hidden without buildkit enabled#2735
silvin-lubecki merged 1 commit into
docker:masterfrom
thaJeztah:fix_flag_hiding

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

These annotations use nil as value, which caused the flag-hiding functions to ignore the annotations, and therefore not hiding the flags.

introduced in #2424

- Description for the changelog

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

These annotations use `nil` as value, which caused the flag-hiding functions
to ignore the annotations, and therefore not hiding the flags.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah

Copy link
Copy Markdown
MemberAuthor

@thaJeztahthaJeztah added this to the 20.10.0 milestone Oct 29, 2020

@chris-cronechris-crone 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

@silvin-lubeckisilvin-lubecki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@silvin-lubecki
silvin-lubecki merged commit 5695d69 into docker:masterOct 29, 2020
@thaJeztah
thaJeztah deleted the fix_flag_hiding branch October 29, 2020 15:24
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@chris-crone@silvin-lubecki