Skip to content

Move tag sanitization after checkout in docker-build workflow#3

Merged
deweysasser merged 2 commits intofix/cleanup-workflowsfrom
copilot/sub-pr-1-again
Feb 18, 2026
Merged

Move tag sanitization after checkout in docker-build workflow#3
deweysasser merged 2 commits intofix/cleanup-workflowsfrom
copilot/sub-pr-1-again

Conversation

Copy link
Contributor

Copilot AI commented Feb 18, 2026

Addresses feedback from PR #1 to reorder steps in the docker-build job for conventional workflow structure.

Changes:

  • Moved Sanitize tag step to run after Checkout step in .github/workflows/build.yaml

Workflow ordering:

docker-build:
  steps:
    - name: Checkout              # Now first
    - name: Sanitize tag          # Moved after checkout
    - name: Login to Registry
    - name: Build Image
    - name: Push

While the sanitization step doesn't require repository files, placing checkout first follows GitHub Actions conventions and improves readability.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…dering

Co-authored-by: deweysasser <1293125+deweysasser@users.noreply.github.com>
@deweysasser deweysasser marked this pull request as ready for review February 18, 2026 05:28
Copilot AI changed the title [WIP] WIP: Address feedback on duplicate checkout steps and cleanup improvements Move tag sanitization after checkout in docker-build workflow Feb 18, 2026
Copilot AI requested a review from deweysasser February 18, 2026 05:30
@deweysasser deweysasser merged commit 3e9c428 into fix/cleanup-workflows Feb 18, 2026
6 checks passed
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