Skip to content

Conversation

@osfameron
Copy link
Collaborator

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the Couchbase Lite C downloads page to use a more maintainable template-based approach instead of hardcoded version-specific content. The change consolidates multiple manually maintained release sections into a streamlined system using parameterized includes.

  • Replaces thousands of lines of hardcoded download tables with parameterized template includes
  • Updates template partial to use standard AsciiDoc [tabs] syntax instead of variable substitution
  • Adds Vector Search 1.0.0 downloads using the same template system

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
modules/c/pages/gs-downloads.adoc Removes hardcoded download tables for versions 3.2.4, 3.2.3, and 3.2.2; replaces with parameterized includes for easier maintenance
modules/c/pages/_partials/downloadslist.adoc Fixes AsciiDoc syntax by removing variable substitution in [tabs] directives

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

[#tbl-downloads-{our-version}]
.Download link table
[{tabs}]
[tabs]
Copy link

Copilot AI Sep 22, 2025

Choose a reason for hiding this comment

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

The change from [{tabs}] to [tabs] is correct. AsciiDoc [tabs] is a standard directive and should not use variable substitution. This improves maintainability by removing unnecessary complexity.

Copilot uses AI. Check for mistakes.
Copy link
Contributor

@iamfortune iamfortune left a comment

Choose a reason for hiding this comment

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

LGTM!

@osfameron osfameron merged commit f48fb47 into release/3.2 Sep 22, 2025
3 checks passed
@osfameron osfameron deleted the 3.2-C-downloads branch September 22, 2025 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants