Skip to content

Add live version to web interface from git tags via Makefile (#61) - #66

Merged
maxi07 merged 1 commit into
devfrom
main
Jul 1, 2026
Merged

maxi07 merged 1 commit into
devfrom
main

Conversation

@maxi07

@maxi07 maxi07 commented Jul 1, 2026

Copy link
Copy Markdown
Owner
  • Initial plan

  • Add dynamic version from git tags via Makefile and Docker build args

  • Create Makefile that reads git tag version via git describe
  • Pass APP_VERSION as Docker build arg through docker-compose.yml
  • Update all Dockerfiles to accept APP_VERSION build arg and set as env var
  • Read APP_VERSION env var in web service instead of hard-coded config value
  • Log app version at web service boot
  • Fall back to config.json version if env var not set

Agent-Logs-Url: https://github.com/maxi07/ScanSync/sessions/1ae7444f-19ab-4386-b174-a6292ff5885d

  • Fix review comments: APP_VERSION override, fallback logic, smb/test service build args

* Initial plan

* Add dynamic version from git tags via Makefile and Docker build args

- Create Makefile that reads git tag version via `git describe`
- Pass APP_VERSION as Docker build arg through docker-compose.yml
- Update all Dockerfiles to accept APP_VERSION build arg and set as env var
- Read APP_VERSION env var in web service instead of hard-coded config value
- Log app version at web service boot
- Fall back to config.json version if env var not set

Agent-Logs-Url: https://github.com/maxi07/ScanSync/sessions/1ae7444f-19ab-4386-b174-a6292ff5885d

Co-authored-by: maxi07 <7480270+maxi07@users.noreply.github.com>

* Fix review comments: APP_VERSION override, fallback logic, smb/test service build args

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: maxi07 <7480270+maxi07@users.noreply.github.com>
@maxi07
maxi07 merged commit 73bfb33 into dev Jul 1, 2026
5 of 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