Skip to content
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

feat: add action to update documentations to yuque #6627

Merged
merged 1 commit into from
Mar 6, 2025
Merged

feat: add action to update documentations to yuque #6627

merged 1 commit into from
Mar 6, 2025

Conversation

yvonneyx
Copy link
Contributor

@yvonneyx yvonneyx commented Mar 6, 2025

  • 添加 GtiHub Action 当PR合并时将自动检测 **/*.md 文件是否变更,有变更将同步到语雀知识库

知识库地址:https://www.yuque.com/antv/luaak6

Copy link

Walkthrough

此PR引入了一个GitHub Action,当PR合并时,自动检测**/*.md文件的变更,并将变更同步到语雀知识库。此功能通过删除旧文档、创建新文档并更新目录来实现。

Changes

文件 概要
.github/workflows/scripts/updateYuque.js 新增脚本,用于处理文档的创建、删除和目录更新。
.github/workflows/update-yuque.yml 新增GitHub Action配置文件。

while (hasMore) {
core.info(`获取文档列表,偏移量: ${offset}, 数量: ${limit}...`);

const response = await fetch(

Choose a reason for hiding this comment

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

The fetch function is used without any error handling for network issues. Consider adding a try-catch block around network requests to handle potential network errors gracefully.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 13693137742

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 86.698%

Totals Coverage Status
Change from base Build 13666991821: 0.0%
Covered Lines: 10605
Relevant Lines: 11857

💛 - Coveralls

@yvonneyx yvonneyx merged commit 086b39e into v5 Mar 6, 2025
2 checks passed
@yvonneyx yvonneyx deleted the action branch March 6, 2025 07:44
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.

3 participants