Skip to content

优化文档中的版本声明周期示例#5920

Open
neveler wants to merge 7 commits intoHMCL-dev:mainfrom
neveler:patch-2
Open

优化文档中的版本声明周期示例#5920
neveler wants to merge 7 commits intoHMCL-dev:mainfrom
neveler:patch-2

Conversation

@neveler
Copy link
Copy Markdown
Contributor

@neveler neveler commented Apr 6, 2026

@neveler
Copy link
Copy Markdown
Contributor Author

neveler commented Apr 6, 2026

顺便看一下这个 HMCL-dev/HMCL-docs#435 (comment)

@Glavo
Copy link
Copy Markdown
Member

Glavo commented Apr 12, 2026

/gemini review

1 similar comment
@Glavo
Copy link
Copy Markdown
Member

Glavo commented Apr 12, 2026

/gemini review

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the release schedule documentation by refining the Mermaid gantt charts across the English, Simplified Chinese, and Traditional Chinese versions. The updates include enabling compact display mode, reordering version sections, and adding specific version milestones along with a current date marker. Feedback was provided to translate the 'today' label in the localized Chinese files to maintain language consistency.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the release schedule documentation across English, Simplified Chinese, and Traditional Chinese versions by adding detailed version milestones and status indicators to the Mermaid Gantt charts. However, the feedback indicates that reversing the section order (listing 3.11 before 3.10 and 3.9) breaks the chart's dependency logic because task IDs are referenced via the "after" keyword before they are defined. Additionally, the use of the non-standard keyword "vert" should be replaced with "milestone" to ensure valid Mermaid syntax.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants