Skip to content

Change notes referencing relevant tests to use the data-test ReSpec feature #62

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

gkellogg
Copy link
Member

@gkellogg gkellogg commented May 31, 2025

The test-manifest HTML generation can extract these from the spec, and generate the HTML with back-references to the relevant part of the spec.

For example, as is done for YAML-LD tests. Note References entry in most test cases that links back to the normative text.

Although a fair amount of work, this can be done with other specs and tests, and becomes valuable for the Implementation Report. Ideally, each test would be referenced by one or more locations in the spec, and each normative statement would reference one or more tests.

(See GitHack rendered version for test details).


Preview | Diff

… feature.

The test-manifest HTML generation can extract these from the spec, and update the HTML with back-references to the relevant part of the spec.

For example, as is done for [YAML-LD tests](https://json-ld.github.io/yaml-ld/tests/manifest.html). Note References entry in most test cases that links back to the normative text.

Although a fair amount of work, this can be done with other specs and tests, and becomes valuable for the Implementation Report. Ideally, each test would be referenced by one or more locations in the spec, and each normative statement would reference one or more tests.
@gkellogg gkellogg added the spec:editorial Minor change in the specification (markup, typo, informative text; class 1 or 2) label May 31, 2025
gkellogg added a commit that referenced this pull request Jun 3, 2025
@gkellogg gkellogg requested review from niklasl, afs, domel and pchampin June 5, 2025 22:23
Copy link
Member

@TallTed TallTed left a comment

Choose a reason for hiding this comment

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

Small stuff

Co-authored-by: Ted Thibodeau Jr <[email protected]>
Co-authored-by: Ted Thibodeau Jr <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec:editorial Minor change in the specification (markup, typo, informative text; class 1 or 2)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants