Skip to content

Conversation

mwichmann
Copy link
Collaborator

All <section> tags in User Guide now have an id="some-tag" (many generated by editor autocomplete). Roughly half had previously been converted. This elminates obscure numeric auto- generated non-constant references in the output that look like:

https://scons.org/doc/production/HTML/scons-user.html#id1514

The old way, any time a section is added, all sections following in the processing order would renumber the auto-generated anchor.

Involves no SCons code changes.

Doc contents don't change either. Added one link to one appendix chapter - but that chapter is not included in the Guide at the moment, Two previously added tags, that were actually linked to, don't follow the naming convention I've been using, so those were adjusted at both the definition and the link location.

Contributor Checklist:

  • I have created a new test or updated the unit tests to cover the new/changed functionality.
  • I have updated CHANGES.txt and RELEASE.txt (and read the README.rst).
  • I have updated the appropriate documentation

All section tags in User Guide now have id="some-tag" (many using editor
autocomplete). Roughly half had previously been converted. This elminates
obscure numeric auto- generated references in the output that look like:

  https://scons.org/doc/production/HTML/scons-user.html#id1514

Signed-off-by: Mats Wichmann <[email protected]>
@mwichmann mwichmann changed the title Add remaing links to User Guide sections [skip appveyor] Add remaining links to User Guide sections Sep 21, 2025
CHANGES.txt Outdated
for known exception types.
- TEMPFILE: Fix the tempfile encoding test regex strings to eliminate
warnings in python 3.12 and 3.13.
- Add tags to the remaining section headers in the User Guide. This is
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like you put this in jcbrill's contributions? Is that an error?
Can you also add to RELEASE.txt?
Otherwise looks good to me.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, good catch. updated.

@bdbaddog bdbaddog merged commit 8074bf1 into SCons:master Sep 24, 2025
8 checks passed
@mwichmann mwichmann added this to 4.10 Sep 24, 2025
@mwichmann mwichmann moved this to Done in 4.10 Sep 24, 2025
@mwichmann mwichmann added this to the 4.10.0 milestone Sep 24, 2025
@mwichmann mwichmann deleted the doc/uguide-tags branch September 24, 2025 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants