Skip to content

Patch workflow#191

Merged
JonayKB merged 5 commits into
mainfrom
PATCH-WORKFLOW
Oct 27, 2025
Merged

Patch workflow#191
JonayKB merged 5 commits into
mainfrom
PATCH-WORKFLOW

Conversation

@JonayKB
Copy link
Copy Markdown
Owner

@JonayKB JonayKB commented Oct 27, 2025

No description provided.

@JonayKB JonayKB self-assigned this Oct 27, 2025
@JonayKB JonayKB added the dependencies Pull requests that update a dependency file label Oct 27, 2025
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modifies the CI/CD workflow to add timestamped Docker image tags alongside the latest tags, enabling version tracking of built images.

Key changes:

  • Adds timestamp generation step to create unique version tags for Docker images
  • Updates both Java API and administration image tagging to include timestamp-based tags

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/dev.yml Outdated
Comment thread .github/workflows/dev.yml Outdated

- name: Create TIMESTAMP
run: TIMESTAMP=$(date +%s)
## CHECK THISSSS
Copy link

Copilot AI Oct 27, 2025

Choose a reason for hiding this comment

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

Remove this TODO comment before merging. Comments like 'CHECK THISSSS' should be resolved and removed from production workflow files.

Suggested change
## CHECK THISSSS

Copilot uses AI. Check for mistakes.
Comment thread .github/workflows/dev.yml Outdated
run: |
ls
ls administration
tags: ${{ vars.DOCKERHUB_USERNAME }}/apiroutinefights:latest , ${{ vars.DOCKERHUB_USERNAME }}/apiroutinefights:$TIMESTAMP
Copy link

Copilot AI Oct 27, 2025

Choose a reason for hiding this comment

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

The TIMESTAMP variable reference should use GitHub Actions syntax ${{ env.TIMESTAMP }} instead of shell variable syntax $TIMESTAMP. Additionally, remove the space after the comma in the tags list as it may cause parsing issues.

Copilot uses AI. Check for mistakes.
Comment thread .github/workflows/dev.yml Outdated
file: administration/Dockerfile
push: true
tags: ${{ vars.DOCKERHUB_USERNAME }}/administrationroutinefights:latest
tags: ${{ vars.DOCKERHUB_USERNAME }}/administrationroutinefights:latest , ${{ vars.DOCKERHUB_USERNAME }}/administrationroutinefights:$TIMESTAMP
Copy link

Copilot AI Oct 27, 2025

Choose a reason for hiding this comment

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

The TIMESTAMP variable reference should use GitHub Actions syntax ${{ env.TIMESTAMP }} instead of shell variable syntax $TIMESTAMP. Additionally, remove the space after the comma in the tags list as it may cause parsing issues.

Copilot uses AI. Check for mistakes.
JonayKB and others added 2 commits October 27, 2025 15:01
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@JonayKB JonayKB requested a review from Copilot October 27, 2025 15:05
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/dev.yml Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@JonayKB JonayKB merged commit 311d2a3 into main Oct 27, 2025
1 check failed
JonayKB added a commit that referenced this pull request Jan 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants