Skip to content

Fix docker-image.yml: indentation, Dockerfile path, and image tag - #2

Draft
jarlungoodoo73 with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-code-comments
Draft

Fix docker-image.yml: indentation, Dockerfile path, and image tag#2
jarlungoodoo73 with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-code-comments

Conversation

Copilot AI commented Aug 4, 2026

Copy link
Copy Markdown

The workflow file had invalid YAML indentation, referenced a non-existent root-level Dockerfile, and used an untraceable timestamp-based image tag.

Changes

  • Indentation — fixed steps entries to be properly nested under steps:
  • Dockerfile pathDockerfile.devcontainer/Dockerfile (the actual Dockerfile in this repo)
  • Image tag$(date +%s)${{ github.sha }} for commit-traceable image tags
# Before
    steps:
    - uses: actions/checkout@v4
    - name: Build the Docker image
      run: docker build . --file Dockerfile --tag my-image-name:$(date +%s)

# After
    steps:
      - uses: actions/checkout@v4
      - name: Build the Docker image
        run: docker build . --file .devcontainer/Dockerfile --tag my-image-name:${{ github.sha }}

…a tag

Co-authored-by: jarlungoodoo73 <201750283+jarlungoodoo73@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix code based on review comments Fix docker-image.yml: indentation, Dockerfile path, and image tag Aug 4, 2026
Copilot AI requested a review from jarlungoodoo73 August 4, 2026 04:49
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.

2 participants