Skip to content

ci: run tests on release-please PRs#90

Closed
aeneasr wants to merge 4 commits intomainfrom
worktree-fix-ci
Closed

ci: run tests on release-please PRs#90
aeneasr wants to merge 4 commits intomainfrom
worktree-fix-ci

Conversation

@aeneasr
Copy link
Copy Markdown
Member

@aeneasr aeneasr commented Apr 2, 2026

Summary

  • Remove github.actor != 'release-please[bot]' guards from all three CI jobs (test, lint, e2e)
  • These conditions caused CI to skip entirely on autorelease PRs, blocking required status checks from passing

Fixes the issue seen in #85 where branch protection checks can't pass.

Test plan

🤖 Generated with Claude Code

aeneasr and others added 4 commits April 2, 2026 20:18
…ustom models

Allows users to run unlisted embedding models (quantized variants, local
fine-tunes, models on non-standard ports) without forking or modifying
KnownModels. Both variables are independently overridable — LUMEN_EMBED_DIMS
is required for unknown models; LUMEN_EMBED_CTX overrides context length for
any model and defaults to 8192 for unlisted ones.

Adds table-driven TestLoad covering: known model (no overrides, each variable
independently, both), unknown model (DIMS only, DIMS+CTX, neither → error,
CTX-only → error).

Documents new env vars and custom model usage in README.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Remove the `github.actor != 'release-please[bot]'` guards so CI jobs
run on autorelease PRs — required for branch protection checks to pass
before merging.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
GitHub suppresses pull_request events when a PR is opened via
GITHUB_TOKEN, so CI never runs on release-please PRs. Passing a PAT
via RELEASE_PLEASE_TOKEN causes the PR to be opened as a real actor,
which does trigger the pull_request event and CI.

Requires a RELEASE_PLEASE_TOKEN secret (PAT with contents:write and
pull-requests:write) to be added to the repo.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@aeneasr
Copy link
Copy Markdown
Member Author

aeneasr commented Apr 2, 2026

Closing — going with the branch protection bypass list approach instead (no code change needed).

@aeneasr aeneasr closed this Apr 2, 2026
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