-
Notifications
You must be signed in to change notification settings - Fork 477
Archive v20.2 documentation #21403
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
Archive v20.2 documentation #21403
Conversation
- Remove LLM-generated openssl_fix.rb script - Remove debug files (error.log, output.txt) - Update sidebar_htmltest.rb to respect htmltest config and skip archived versions This addresses issues similar to those found in PR #21195 for v2.1: - Removes SSL workaround script that doesn't belong in src/current - Removes accidentally committed build artifacts - Ensures htmltest skips archived versions during validation
✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.
|
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
Files changed:
|
✅ Netlify Preview
To edit notification comments on pull requests, go to your Netlify project configuration. |
rmloveland
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| gem "webrick" | ||
| gem "jekyll-minifier" | ||
|
|
||
| gem "csv" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gemfile changes should go in a separate PR
| @site = site | ||
|
|
||
| # Read htmltest configuration to get ignored directories | ||
| htmltest_config = YAML.load_file('.htmltest.yml') rescue {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rmloveland
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ebembi-crdb this LGTM once you implement the changes I mentioned in the comments. Wanted to unblock you on this since I will be on PTO for the holidays next week
- Add v20.2 offline documentation link to archived documentation page - Remove v20.2 specific files from repository including: - src/current/v20.2/ directory - src/current/_includes/sidebar-data-v20.2.json - src/current/images/v20.2/ directory - Resolve merge conflicts with main branch - Revert Gemfile changes per review feedback
The build requires jekyll-minifier gem. Will be removed in separate PR as requested in review.
- Resolved merge conflicts in archived-documentation.md - Removed v20.2/cockroach-start.md (archiving v20.2) - Fixed broken links to v20.2 topology documentation
- Resolved merge conflicts by confirming deletion of v20.2 files - Fixed broken v20.2 geospatial image references by updating to each version's own images - Preserved v21.1 documentation files on this branch - Updated archived-documentation.md to include both v21.1 and v20.2
Add missing destination-file-privileges.md include file for v21.1 by copying from v21.2 to fix build error
Resolved conflicts by: - Keeping v21.2 documentation files on this branch (only archiving v20.2) - Not adding v21.2 to archived-documentation.md (v21.2 still active on this branch) - Accepting deletion of v21.2 release include files from main
Add missing diagram files for v21.1 documentation: - backup.html - restore.html These files are required by v21.1/backup.md and v21.1/restore.md Copied from v20.2/sql/diagrams to v21.1/sql/generated/diagrams
v21.1 was already archived on main, so it should also be deleted on this branch to avoid build failures. This replaces the incorrect approach of creating missing v21.1 includes. Following the proper archival pattern from successful PRs: - Delete ALL files for archived versions - Don't patch other versions with missing includes
v21.2 was already archived on main, so it should also be deleted on this branch to avoid build failures with missing includes. Following the proper archival pattern: - Delete ALL files for archived versions - Don't patch other versions with missing includes
Resolved merge conflicts: - Deleted v22.2 files that were already archived on main - Updated archived-documentation.md to include v22.2 and v21.2 entries - Deleted v21.2 files to avoid build failures (already archived on main)
Resolved merge conflicts: - Deleted v22.1 files that were already archived on main - Removed conflicting v22.1 includes and documentation files
Archives v20.2 documentation following the same process used for v2.1.
Changes