Skip to content

chore(release): force tag fetch in release-portlet.sh preflight#443

Open
bjagg wants to merge 1 commit into
uPortal-Project:masterfrom
bjagg:chore/harden-release-fetch
Open

chore(release): force tag fetch in release-portlet.sh preflight#443
bjagg wants to merge 1 commit into
uPortal-Project:masterfrom
bjagg:chore/harden-release-fetch

Conversation

@bjagg

@bjagg bjagg commented Jul 2, 2026

Copy link
Copy Markdown
Member

Problem: the tree-state git fetch upstream <branch> --tags had no --force, so a stale local tag (one re-cut on upstream during an earlier release) could exit non-zero and — under set -euo pipefailsilently abort the whole script right after the branch check. (This bit the uPortal release; the portlet scripts share the same line.)

Goal: don't let stale local tags abort a release.

Changes (release-portlet.sh):

  • add --force to the tree-state tag fetch.

Notes: mirrors uPortal #3005 (release-uportal.sh). This Maven script does not need #3005's push-branch fix — maven-release-plugin already pushes master to upstream via scm.developerConnection. bash -n clean.

Problem: the tree-state `git fetch upstream <branch> --tags` had no
`--force`, so a stale local tag (one re-cut on upstream during an earlier
release) could exit non-zero and, under `set -euo pipefail`, silently
abort the whole script right after the branch check.

Goal: don't let stale local tags abort a release.

Changes:
- release-portlet.sh: add `--force` to the tree-state tag fetch.

Notes: mirrors uPortal PR #3005 (release-uportal.sh). This Maven script
does NOT need #3005's push-branch fix — maven-release-plugin already
pushes master to upstream via scm.developerConnection. bash -n clean.
@bjagg bjagg requested a review from Naenyn July 2, 2026 23:05
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