Skip to content

Conversation

@tomchon
Copy link
Contributor

@tomchon tomchon commented Oct 6, 2025

fix: remove unnecessary jsonPath attribute and add tdgpt download url

Copy link
Contributor

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 removes unnecessary jsonPath attributes from PkgListV37 components and updates the TDgpt tutorial with specific download information and package details.

  • Removes redundant jsonPath parameters from PkgListV37 components
  • Adds specific TDgpt package component with version 3.3.8.0 details
  • Updates installation commands with concrete package names instead of placeholders

Reviewed Changes

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

File Description
docs/zh/06-advanced/06-TDgpt/02-tutorial.md Adds TDgpt-specific package component and replaces generic installation commands with concrete package names
docs/zh/04-get-started/03-package.mdx Removes unnecessary jsonPath attributes from existing PkgListV37 components

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

description: 使用 docker、云服务、安装包体验 TDgpt
---

import PkgListV3 from "/components/PkgListV3";
Copy link

Copilot AI Oct 6, 2025

Choose a reason for hiding this comment

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

The new import PkgListV37 appears to be unused since the old PkgListV3 component is being replaced rather than used alongside it. Consider removing the unused import PkgListV3 on line 7 if it's no longer needed.

Suggested change
import PkgListV3 from "/components/PkgListV3";

Copilot uses AI. Check for mistakes.
@feici02 feici02 merged commit cad8d4d into main Oct 6, 2025
5 checks passed
@feici02 feici02 deleted the docs/fix-download-url branch October 6, 2025 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants