Block a user
ci: add SOP checklist gate
LGTM. SOP checklist gate is well-implemented. Trust boundary (pull_request_target + BASE ref checkout) is correct. Tier-aware and idempotent.
plugin-dev
commented on pull request molecule-ai/molecule-ai-plugin-ecc#10
2026-05-14 12:56:34 +00:00
chore: add gitea-merge-queue.yml workflow
plugin-dev review
Overall: LGTM. The script is well-structured, defensive, and the serialized one-PR-per-run design is sound.
Observations
**1. AUTO_SYNC_TOKEN secret not yet…
chore: add gitea-merge-queue.yml workflow
LGTM — same code as MCP server PR#13/canonical version. Same observations:
fix: clarify test deps in CLAUDE.md
LGTM — clarifying that pip install -e . only installs base packages and pip install -e '.[test]' is needed for the full test suite is a useful doc fix. Many contributors would miss the pytest-asyncio optional dependency without this guidance. Ship it.
chore: add gitea-merge-queue.yml workflow
LGTM — well-designed merge queue script. Key observations: