Exclude archived FAQ content from Jekyll build; clean up archive references - #55
Merged
Merged
Conversation
- barcode-reader/server/archive/unable-to-scan-aztec-code.md duplicated the live web-edition page (barcode-reader/web/scan-setting/unable-to-scan-aztec-code.md) and didn't belong in the server FAQ. Removed the file and its entry in server/configuration/index.md. - barcode-reader/web/archive/use-of-onFrameRead-and-onUnduplicateRead.md stays in the repo as historical/archived content, but is no longer linked from web/configuration/index.md. Its filename typo (onUnduplicateRead vs onUniqueRead) isn't worth fixing now that it's unlinked archived content. Closes #41, closes #42 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
barcode-reader/{mobile,server,web}/archive/ hold historical,
version-pinned content (e.g. v9.6.40, v10.4.2000 snapshots) that's
no longer linked from any live index or sidebar. Excluding them from
the build means they're kept in the repo for reference but are no
longer built, hosted, or listed in sitemap.xml.
Verified locally with `jekyll build`: the three directories are
absent from the generated _site output and from sitemap.xml, and the
two edited index pages (server/configuration, web/configuration)
render with no dangling references.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
lyzhang0113
marked this pull request as ready for review
August 4, 2026 09:09
This was referenced Aug 4, 2026
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Follow-up to the discussion on #30 about
*/archive/*content: these directories hold historical, version-pinned FAQ snapshots (e.g.-v9.6.40,-v10.4.2000) that are no longer linked from any live index or sidebar. Decision: keep the files in the repo for reference, but stop building/hosting/sitemap-listing them. Also folds in a related cleanup: README.md was a Jekyll page duplicating the homepage and unlinked from anywhere, so it's now a plain contributor README, also excluded from the build.barcode-reader/server/archive/unable-to-scan-aztec-code.mdand its entry inserver/configuration/index.md— it duplicated the live web-edition page (barcode-reader/web/scan-setting/unable-to-scan-aztec-code.md) and didn't belong in the server FAQ. Fixes server/configuration/index.md: Aztec-code archive entry may not belong in server FAQ #41.barcode-reader/web/archive/use-of-onFrameRead-and-onUnduplicateRead.mdfromweb/configuration/index.md, but kept the file itself as archived content. Its filename typo (onUnduplicateReadvsonUniqueRead) isn't worth fixing now that it's unlinked. Closed Filename typo: onUnduplicateRead should be onUniqueRead #42 (won't-fix, not a real fix).exclude:in_config.ymlforbarcode-reader/mobile/archive,barcode-reader/server/archive, andbarcode-reader/web/archive. Verified locally with an actualjekyll build: all three directories are absent from the generated_siteoutput and fromsitemap.xml, and both edited index pages render with no dangling links.README.mdas a plain repo-contributor README (structure, build/deploy process, link-checking tool) instead of a Jekyll page, and added it to_config.yml'sexclude:list. Verified locally that it no longer appears in the build output orsitemap.xml. Fixes README.md is an unlinked duplicate of the homepage, not excluded from the site build #53.Related issues — already closed as won't-fix
These were all filed against content inside
barcode-reader/mobile/archive/. Once this PR merges, none of that content is built or reachable on the live site, so the broken links / stranded content / unresolved variable they describe have no user-facing impact. Already closed (won't-fix,wontfixlabel) ahead of merge:Test plan
jekyll buildconfirms archive dirs and README.md are excluded from_siteandsitemap.xml🤖 Generated with Claude Code