Skip to content

Add github Ci action for tagging every merged PR - #287

Draft
furtib wants to merge 1 commit into
Ericsson:mainfrom
furtib:auto-tag-every-commit
Draft

Add github Ci action for tagging every merged PR#287
furtib wants to merge 1 commit into
Ericsson:mainfrom
furtib:auto-tag-every-commit

Conversation

@furtib

@furtib furtib commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Why:
We want to tag every commit made to main for easier integration and testing for end users.

What:

  • Added a GitHub Action that creates a tag for every merged PR.
    Use the date and time for the tag name.

Addresses:
#285

@furtib
furtib requested a review from Szelethus August 13, 2026 11:37
@furtib furtib self-assigned this Aug 13, 2026
@furtib furtib added the CI 📦 label Aug 13, 2026

@Szelethus Szelethus left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

jobs:
tag:
if: github.event.pull_request.merged == true
runs-on: ubuntu-latest

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Do we even need to specify this?

@furtib
furtib marked this pull request as draft August 13, 2026 12:56

- name: Create and push tag
run: |
TAG_NAME="v$(date +%Y.%m.%d.%H%M%S)"

@nettle nettle Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is not good idea I think...
Tag as a version number is good, tag as date is not much better than commit hash.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants