Skip to content

fix: resolve issue #14 - First contribution: add a short glossary for the existing validator - #31

Open
nazasnow wants to merge 1 commit into
borek504:mainfrom
nazasnow:fix-issue-14-2249
Open

nazasnow wants to merge 1 commit into
borek504:mainfrom
nazasnow:fix-issue-14-2249

Conversation

@nazasnow

Copy link
Copy Markdown

Closes #14

Summary

Copy link
Copy Markdown
Owner

Thanks for the contribution. I’m keeping #31 as the single canonical PR for #14 and closed the earlier duplicate PRs #17–#30 so the review has one source of truth.

I checked the current #31 diff. The glossary text is broadly aligned with the issue, but the PR is not merge-ready yet because #14 explicitly requires both:

  • docs/glossary.md with the six existing v1 terms, and
  • one relative link from README’s Design boundary section.

Please update #31 only (do not open another PR) to add that README link. Also add the final newline to docs/glossary.md. Keep the scope documentation-only: no Python, CI, schema, release, or Result Envelope v2 terminology.

The important trust-boundary wording should remain explicit: digests bind contents but are not signatures/proof of truth; attempt_id/unit_id are identifiers, not credentials; valid is not authorization.

Once that small delta is pushed, we can review this one PR normally.

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.

First contribution: add a short glossary for the existing validator

2 participants