Skip to content

[codex] Remove PrimeCert bookkeeping - #228

Merged
Vilin97 merged 2 commits into
mainfrom
codex/remove-primecert-bookkeeping
Jun 26, 2026
Merged

Vilin97 merged 2 commits into
mainfrom
codex/remove-primecert-bookkeeping

Conversation

@Vilin97

@Vilin97 Vilin97 commented Jun 26, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Finish the non-content cleanup after removing PrimeCert from Lean Pool content in #226.

This removes the stale NOTICE attribution and candidate-table/manual-list entries, and changes the candidate decision for b-mehta/primecert to include:false because the upstream project is actively maintained.

This also fixes the newly added project-advisory workflow so PRs with no added or changed LeanPool/projects.yml entries skip comment creation instead of failing while trying to post an empty advisory.

CI scope

This PR is non-content-only, so it is intentionally separate from #226 to satisfy content-pr-guard.yml.

Validation

  • cd python && uv run --group test pytest tests/test_pr_advisory.py
  • cd python && uv run ruff check lean_pool/pr_advisory.py tests/test_pr_advisory.py
  • cd python && uv run ruff format --check lean_pool/pr_advisory.py tests/test_pr_advisory.py
  • cd python && uv run python -m lean_pool.pr_advisory --pr-number 228 --dry-run
  • Parsed candidates/decisions.jsonl locally
  • rg -n "PrimeCert|primecert|Primecert" . now only finds the explicit include:false candidate decision

@Vilin97
Vilin97 marked this pull request as ready for review June 26, 2026 16:35
Copilot AI review requested due to automatic review settings June 26, 2026 16:35

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Vilin97
Vilin97 merged commit dcea321 into main Jun 26, 2026
3 of 4 checks passed
@Vilin97
Vilin97 deleted the codex/remove-primecert-bookkeeping branch June 26, 2026 16:40
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