Skip to content

Add GitHub Actions workflow for auto-tagging and notifications#7

Merged
sachintu47 merged 4 commits into
zopen2from
sachintu47-patch-1
May 20, 2026
Merged

Add GitHub Actions workflow for auto-tagging and notifications#7
sachintu47 merged 4 commits into
zopen2from
sachintu47-patch-1

Conversation

@sachintu47
Copy link
Copy Markdown
Member

@sachintu47 sachintu47 commented May 20, 2026

Auto-tag on every release and notify downstream zoslibport on updates

Signed-off-by: Sachin <32639496+sachintu47@users.noreply.github.com>
@sachintu47
Copy link
Copy Markdown
Member Author

augment review

@augmentcode
Copy link
Copy Markdown

augmentcode Bot commented May 20, 2026

🤖 Augment PR Summary

Summary: Adds a GitHub Actions workflow to automate version tagging/releases and notify a downstream repo.

Changes:

  • Introduces .github/workflows/auto-tag-notify.yml triggered on merged PRs into zopen2 (pull_request_target) and on published releases
  • Checks out the merge commit (PR path) or the release tag (release path)
  • On PR merges, bumps the tag and creates a GitHub Release with autogenerated notes
  • Dispatches a repository_dispatch event to zopencommunity/zoslibport to notify downstream of updates

Technical Notes: Actions are pinned to immutable SHAs and the workflow requests contents: write to allow tag/release creation.

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode Bot left a comment

Choose a reason for hiding this comment

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

Review completed. 3 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Comment thread .github/workflows/auto-tag-notify.yml Outdated
Comment thread .github/workflows/auto-tag-notify.yml Outdated
Comment thread .github/workflows/auto-tag-notify.yml
Updated workflow to use pull_request_target for better security and modified actions to use specific commit SHAs.

Signed-off-by: Sachin <32639496+sachintu47@users.noreply.github.com>
@sachintu47
Copy link
Copy Markdown
Member Author

augment review

Copy link
Copy Markdown

@augmentcode augmentcode Bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestion posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Comment thread .github/workflows/auto-tag-notify.yml
Add conditional ref for checkout based on event type.

Signed-off-by: Sachin <32639496+sachintu47@users.noreply.github.com>
@sachintu47
Copy link
Copy Markdown
Member Author

augment review

Copy link
Copy Markdown

@augmentcode augmentcode Bot left a comment

Choose a reason for hiding this comment

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

Review completed. 2 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Comment thread .github/workflows/auto-tag-notify.yml
Comment thread .github/workflows/auto-tag-notify.yml
Signed-off-by: Sachin <32639496+sachintu47@users.noreply.github.com>
@sachintu47 sachintu47 marked this pull request as ready for review May 20, 2026 09:47
@sachintu47 sachintu47 merged commit 51e835f into zopen2 May 20, 2026
2 checks passed
Copy link
Copy Markdown

@augmentcode augmentcode Bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestion posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

pull_request_target:
types: [closed]
branches: [zopen2]
release:
Copy link
Copy Markdown

@augmentcode augmentcode Bot May 20, 2026

Choose a reason for hiding this comment

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

Because this workflow also triggers on release: published, a merge run that creates a release will cause a second run and likely dispatch upstream-release twice. Please confirm the downstream can handle duplicate notifications (or has a dedupe mechanism) so this doesn’t trigger redundant downstream work.

Severity: medium

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

@sachintu47 sachintu47 deleted the sachintu47-patch-1 branch May 27, 2026 13:54
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.

1 participant