Skip to content

chore(master): release 1.5.0#15

Merged
mlorentedev merged 2 commits into
masterfrom
release-please--branches--master
Mar 8, 2026
Merged

chore(master): release 1.5.0#15
mlorentedev merged 2 commits into
masterfrom
release-please--branches--master

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented Mar 8, 2026

🤖 I have created a release beep boop

1.5.0 (2026-03-08)

Features

  • graceful drain and multi-package refactoring (#14) (14355b7)

This PR was generated with Release Please. See documentation.

@mlorentedev mlorentedev merged commit 06e9cd8 into master Mar 8, 2026
11 checks passed
@mlorentedev mlorentedev deleted the release-please--branches--master branch March 8, 2026 04:07
@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions Bot commented Mar 8, 2026

🤖 Created releases:

🌻

mlorentedev added a commit that referenced this pull request May 18, 2026
## Summary

PRs opened by the default `GITHUB_TOKEN` do not trigger `pull_request`
workflows — GitHub suppresses these events to prevent infinite recursion
when one workflow creates a PR that would trigger another. With branch
protection on `master` requiring `test` / `lint` / `security` status
checks, the auto-generated `release-please` PR ends up in `BLOCKED`
state with no CI ever running.

This is what hit us today on PR #19 (v1.5.1). The previous release PR
#15 (v1.5.0) silently dodged the issue because something retriggered CI
manually 11 minutes after the PR opened (likely a UI "Update branch"
click).

## Fix

Pass an explicit PAT to `release-please-action` via the `token:` input.
PRs created with a PAT appear as user-driven events to GitHub, so the
`pull_request` trigger fires normally.

```yaml
token: ${{ secrets.RELEASE_PLEASE_PAT || secrets.GITHUB_TOKEN }}
```

The `||` fallback to `GITHUB_TOKEN` means this PR is safe to merge
**before** the secret is configured — the workflow keeps using the
legacy behavior until the PAT is added. No disruption.

## Setup checklist (one-time, after merging this PR)

- [x] Generate a [fine-grained
PAT](https://github.com/settings/personal-access-tokens/new):
  - **Repository access:** Only select `mlorentedev/ts-bridge`
  - **Permissions:**
    - `Contents`: Read and write
    - `Pull requests`: Read and write
- `Workflows`: Read and write *(only if a release PR ever changes a
`.github/workflows/*` file — uncommon)*
  - **Expiration:** 1 year recommended, with calendar reminder
- [x] Add it as repo secret `RELEASE_PLEASE_PAT` (Settings → Secrets and
variables → Actions → New repository secret)
- [x] On the next release PR, verify CI fires automatically (no
close+reopen dance)

## Alternatives considered (rejected)

| Option | Why rejected |
|---|---|
| Switch `ci.yml` to `pull_request_target` | Exposes base-branch secrets
to PR code — unsafe surface for `test`/`lint` jobs |
| Empty commit / close+reopen on each release | Manual toil per release;
defeats the point of automation |
| Drop branch protection on `master` | Loses the safety net for human
PRs too |
| Use a dedicated GitHub App | Heavier setup; PAT is sufficient for a
single-maintainer repo |

## Test plan

- [x] `go vet ./...` clean (no Go changes, just the workflow)
- [x] CI on this PR runs the standard checks
- [x] After merge + secret setup, manually run release.yml on master or
wait for the next `fix:`/`feat:` to verify the next release PR has CI
checks attached at creation time
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant