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

call input to set method for evaluating build #1265

Merged
merged 2 commits into from
Nov 25, 2024

Conversation

crazy-max
Copy link
Member

@crazy-max crazy-max commented Nov 25, 2024

Main case for this input is to be able to validate the build configuration before building using the check call as such:

name: ci

on:
  push:

jobs:
  docker:
    runs-on: ubuntu-latest
    steps:
      -
        name: Set up Docker Buildx
        uses: docker/setup-buildx-action@v3
      -
        name: Login to Docker Hub
        uses: docker/login-action@v3
        with:
          username: ${{ secrets.DOCKERHUB_USERNAME }}
          password: ${{ secrets.DOCKERHUB_TOKEN }}
      -
        name: Validate build configuration
        uses: docker/build-push-action@v6
        with:
          call: check
      -
        name: Build and push
        uses: docker/build-push-action@v6
        with:
          push: true
          tags: user/app:latest

image

@crazy-max

This comment was marked as resolved.

@crazy-max crazy-max force-pushed the call-check branch 2 times, most recently from 16616d6 to f78ddda Compare November 25, 2024 17:26
Copy link
Contributor

@dvdksn dvdksn left a comment

Choose a reason for hiding this comment

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

❤️

@thompson-shaun thompson-shaun merged commit 0259cb0 into docker:master Nov 25, 2024
63 checks passed
@crazy-max crazy-max deleted the call-check branch November 25, 2024 19:56
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.

4 participants