Skip to content

no-cache-filters input#653

Merged
crazy-max merged 1 commit intodocker:masterfrom
crazy-max:no-cache-filters
Jul 19, 2022
Merged

no-cache-filters input#653
crazy-max merged 1 commit intodocker:masterfrom
crazy-max:no-cache-filters

Conversation

@crazy-max
Copy link
Member

fixes #645

Signed-off-by: CrazyMax crazy-max@users.noreply.github.com

@crazy-max crazy-max requested review from jedevc and tonistiigi July 18, 2022 15:28
@jedevc
Copy link
Contributor

jedevc commented Jul 18, 2022

See comment here

Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
@crazy-max
Copy link
Member Author

See comment here

Yes indeed, not sure myself as we currently make inputs plural in this action like outputs.

@jedevc
Copy link
Contributor

jedevc commented Jul 19, 2022

we currently make inputs plural in this action like outputs

Ah sorry, I got confused. Makes sense to allow this field as this then, but long-term may be worth discussing about option consistency throughout actions/buildx/bake.

@crazy-max crazy-max merged commit 1cb9d22 into docker:master Jul 19, 2022
@crazy-max crazy-max deleted the no-cache-filters branch July 19, 2022 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature request: Add no-cache-filter

2 participants