Skip to content

chore(main): release 7.1.1 - #163

Open
github-actions[bot] wants to merge 1 commit into
mainfrom
release-please--branches--main
Open

github-actions[bot] wants to merge 1 commit into
mainfrom
release-please--branches--main

Conversation

@github-actions

@github-actions github-actions Bot commented Sep 15, 2026 •

Copy link
Copy Markdown
Contributor

🤖 I have created a release beep boop

7.1.1 (2026-09-25)

Bug Fixes

  • ci: run the suite before tagging a release (#162) (3594ac9)

This PR was generated with Release Please. See documentation.

@github-actions
github-actions Bot requested a review from tbarbugli as a code owner September 15, 2026 17:10
@github-actions
github-actions Bot force-pushed the release-please--branches--main branch from d6b4342 to cfe9163 Compare September 17, 2026 12:22
@mogita
mogita deployed to feeds-enabled-shard September 17, 2026 15:04 — with GitHub Actions Active
@mogita
mogita deployed to feeds-enabled-shard September 17, 2026 15:04 — with GitHub Actions Active
@mogita
mogita deployed to feeds-enabled-shard September 17, 2026 15:04 — with GitHub Actions Active
@mogita
mogita deployed to feeds-enabled-shard September 17, 2026 15:04 — with GitHub Actions Active
@mogita
mogita deployed to feeds-enabled-shard September 17, 2026 15:04 — with GitHub Actions Active
@mogita
mogita deployed to feeds-enabled-shard September 17, 2026 15:04 — with GitHub Actions Active
mogita added a commit that referenced this pull request Sep 17, 2026
The github.actor clause made the skip miss the path CONTRIBUTING tells
maintainers to take. With strict: true the Release PR has to be brought up to
date whenever main moves, and Update branch attributes that merge commit to
the person who clicked, so github.actor is a human and the guard fell through
to running the full suite. Verified on #163: 8c31d76 is exactly that merge,
and its run had actor=mogita and spent 3m51s on the 1.19 integration leg.

Dropping the clause fires the skip on both Update branch and Approve and run.
The remaining three clauses still cannot be forged: a PR author cannot be set
to github-actions[bot], and the head repository must be this one.

The cost is that a human commit pushed onto a Release PR is no longer tested
here. release.yml calls this same workflow against the merge commit before it
tags, which is what actually gates the release, and a `run-tests` label now
forces the suite on any Release PR, since a re-run keeps the original actor
and neither workflow has a dispatch trigger.

Also corrects two comments. The claim that lint.yml and reviewdog.yml compile
version.go was wrong on both counts: lint.yml only runs `go mod tidy`, and
reviewdog.yml pipes golangci-lint into reviewdog under `bash -e` without
pipefail, so the job takes reviewdog's exit status and its default
-fail-level=none returns 0. And the note that a job-level `if:` is evaluated
before the matrix expands is restored, because it is the only record of why
re-adding the six required contexts would block every Release PR forever.
@github-actions
github-actions Bot force-pushed the release-please--branches--main branch 2 times, most recently from fd46666 to b745cd2 Compare September 24, 2026 09:09
@github-actions
github-actions Bot force-pushed the release-please--branches--main branch from b745cd2 to 0b1e4ee Compare September 25, 2026 10:45

This branch was successfully deployed

1 active (outdated) deployment
feeds-enabled-shard — 8c31d763 Deployed Sep 17, 2026 by mogita via ci / 👷 Test & Build 1.20 #545
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