Skip to content

Switch makedoc.g to using AutoDoc - #53

Open
fingolfin wants to merge 1 commit into
masterfrom
mh/autodoc
Open

Switch makedoc.g to using AutoDoc#53
fingolfin wants to merge 1 commit into
masterfrom
mh/autodoc

Conversation

@fingolfin

Copy link
Copy Markdown
Member

The hand written ExtractMyManualExamples in makedoc.g produced test files
which did not actually pass when running the test suite. Use AutoDoc for
extracting the examples instead, as it is done in several other packages
by now.

The version data entities move from a GAPDoc block in PackageInfo.g into
the AutoDoc entities record, together with the entities for the other
packages referred to by the manual; from these AutoDoc generates
doc/_entities.xml. The entity for SCSCP itself is added by AutoDoc.

AutoDoc numbers the generated test files consecutively, while the old
code used the chapter number, so tst/scscp0{4,...,9}.tst become
tst/scscp0{1,...,6}.tst. Adjust the list of files to exclude in
tst/testall.g accordingly, as those names are hardcoded there.

Co-Authored-By: Claude Opus 5 noreply@anthropic.com

The hand written ExtractMyManualExamples in makedoc.g produced test files
which did not actually pass when running the test suite. Use AutoDoc for
extracting the examples instead, as it is done in several other packages
by now.

The version data entities move from a GAPDoc block in PackageInfo.g into
the AutoDoc entities record, together with the entities for the other
packages referred to by the manual; from these AutoDoc generates
doc/_entities.xml. The entity for SCSCP itself is added by AutoDoc.

AutoDoc numbers the generated test files consecutively, while the old
code used the chapter number, so tst/scscp0{4,...,9}.tst become
tst/scscp0{1,...,6}.tst. Adjust the list of files to exclude in
tst/testall.g accordingly, as those names are hardcoded there.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@codecov

codecov Bot commented Jul 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.53%. Comparing base (7f4040d) to head (c824b38).

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #53   +/-   ##
=======================================
  Coverage   69.53%   69.53%           
=======================================
  Files          24       24           
  Lines        2669     2669           
=======================================
  Hits         1856     1856           
  Misses        813      813           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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