Skip to content

Release 1.0.0 - #3

Merged
pjhartlieb merged 7 commits into
stablefrom
dev
Oct 29, 2026
Merged

pjhartlieb merged 7 commits into
stablefrom
dev

Conversation

@pjhartlieb

Copy link
Copy Markdown
Contributor

Promotes dev to stable for the first release, v1.0.0.

On merge, release.yml runs release-check and the tests, then waits twice at the release environment (credentials, then publish). Each approval needs a bbot-leads member other than whoever triggered the run. bls-release then tags v1.0.0 on the merge commit.

pjhartlieb and others added 7 commits October 28, 2026 15:03
Remove the agreement, signatures and the Dependabot auto-merge workflow (D10);
they stay in CLA or are retired. Rename the project to release-workflows.

Add release.yml (release this repo from stable with its own workflows) and
dev-security.yml (dependency review, actionlint, zizmor). Scope the CLA App
token to CLA and the calling repository with the permissions it uses.
Dependabot cooldown of 7 days; self-repository syntax for local calls.
The caller's pull request commits are read with its own GITHUB_TOKEN, so the App token no longer needs the calling repository or write access to it.
CodeQL default setup was enabled after the last push; reopening the pull request did not start a scan.
The code scanning rule was added to dev-review after the previous push; reopening the pull request did not start a scan.
@pjhartlieb
pjhartlieb requested a review from a team as a code owner October 29, 2026 13:56

@en0f en0f left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved. Two things to confirm only:

1.) The BLS_RELEASE_APP_ID variable and BLS_RELEASE_PRIVATE_KEY secret are set on the release environment, and that the release App is installed on this repo

2.) that the CLA App has members:read and is installed on the CLA repo, now that its token asks for those permissions explicitly.

@pjhartlieb
pjhartlieb merged commit 5c7ee0e into stable Oct 29, 2026
25 checks passed
@pjhartlieb

Copy link
Copy Markdown
Contributor Author

Both confirmed.

  1. The release environment has the variable BLS_RELEASE_APP_ID (5401785) and the secret BLS_RELEASE_PRIVATE_KEY. The release App is installed on selected repositories with contents write and metadata read only. Its repository list isn't readable from here, so the credentials job in the Release run is what proves it covers this repo.
  2. The CLA App is installed on all repositories, so CLA is included, with members: read and contents: write. That matches what cla-reusable.yml requests.

The Release run is waiting at release (credentials). Self-review prevention stops whoever merged from approving it, so it needs another bbot-leads member.

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