Skip to content

docs: checklist names the published public repo, refresh demo measurement - #76

Merged
FetchDoge merged 1 commit into
mainfrom
docs/checklist-public-repo
Nov 4, 2026
Merged

FetchDoge merged 1 commit into
mainfrom
docs/checklist-public-repo

Conversation

@FetchDoge

Copy link
Copy Markdown
Owner

The submission checklist lives in the published repo, but its Public code repository section said this checkout's git remote was None and that publishing was not done here. A judge reading the checklist on GitHub was told the page they were reading does not exist.

Changes (docs + one test)

  • docs/submission-checklist.md: Public code repository section now names the public repo, https://github.com/FetchDoge/permit, as published, with that URL as the Devpost repo field (the form itself is still filled at submission time). Verified 2026-11-04: the LICENSE file reads back via the API as Apache-2.0 and main is at 33a3694.
  • Functional demo measurement refreshed from 2026-10-14 to 2026-11-04 on main at 33a3694.
  • tests/test_submission_docs.py: new test_checklist_names_public_repo pins the URL and the absence of both stale claims.

Evidence (local, 2026-11-04, base tree 33a3694)

  • Full suite: 425 passed, 3 skipped.
  • python3 demo.py: ledger chain VERIFIED, 16 receipts.
  • Six-beat replay: VERIFIED, 26 receipts, 0.72s, matching the refreshed measurement line.
  • New test: 6 passed in test_submission_docs (5 baseline + 1 new); 14 passed under python3 -O for test_submission_docs + test_sandbox_smoke.
  • No em dashes added.

…ment (#76)

The Public code repository section claimed this checkout's git
remote was None and that publishing was not done here. The repo is
public at https://github.com/FetchDoge/permit (Apache-2.0 LICENSE
reads back live, main at 33a3694, verified 2026-11-04), and the
checklist itself is read by judges in that repo. Section now names
the public URL as the Devpost repo field, with the form itself
still filled at submission time.

Functional demo measurement refreshed from 2026-10-14 to
2026-11-04 on main at 33a3694: six-beat replay VERIFIED,
26 receipts, 0.72s locally; full suite 425 passed, 3 skipped
on the base tree today.

New test test_checklist_names_public_repo pins the URL and the
absence of both stale claims. Docs and one test only.

Co-authored-by: Marcus Richards <marsoju@gmail.com>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-04T15:33:00.300415Z ae5cb9e PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@FetchDoge
FetchDoge merged commit edc372c into main Nov 4, 2026
9 checks passed
@FetchDoge
FetchDoge deleted the docs/checklist-public-repo branch November 4, 2026 15:33
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