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

chore(sync): Sitemap Creator 1.0.4 #10

Merged
merged 14 commits into from
Feb 7, 2025
Merged

chore(sync): Sitemap Creator 1.0.4 #10

merged 14 commits into from
Feb 7, 2025

Conversation

DuckDuckStudio
Copy link
Owner

检查清单

  • 有链接Issue吗? -- Resolve
  • 你检查过没有其他重复的 拉取请求 吗?
  • 此拉取请求仅针对一个问题/功能吗?
  • 你验证过你的修改吗?
  • 你确定你的描述足以让开发人员理解你的意图以及解决方案?
  • 你知道你现在在 稳定版 仓库中

修改说明


更多信息:
fix: 修复了当出现“未知更新方式”错误时显示的当前设置错误的问题 (其实就是用错环境变量了,用到自动更新那个去了)
fix: 修复基础分支定义但未使用的问题
更多信息:
fix?: 修复在Windows/Macos Runner上设置时区的问题
因为它们只是 DEBUG 日志而不是警告。
为什么不使用 console.debug()。此方法仅在日志等级为 Debug 时才会输出,而我们目前的控制方式是环境变量中的 Debug 参数而不是日志等级。
匹配逻辑:
const outdatedPRs = pulls.filter(pr => pr.title.includes('自动更新网站地图') && pr.base.ref === process.env.BASE_BRANCH && pr.head.ref.includes('Sitemap_Creator'));
PR标题中带有“自动更新网站地图”
PR基分支是参数中设置的基分支
PR头分支名中包含“Sitemap_Creator”
@DuckDuckStudio DuckDuckStudio added DEV-开发分支合并 开发分支的合并PR DEV-测试通过 通过测试 labels Feb 7, 2025
@DuckDuckStudio DuckDuckStudio added this to the 1.0.4 milestone Feb 7, 2025
@fjwxzde fjwxzde linked an issue Feb 7, 2025 that may be closed by this pull request
index.mjs Fixed Show fixed Hide fixed
index.mjs Fixed Show fixed Hide fixed
@fjwxzde fjwxzde linked an issue Feb 7, 2025 that may be closed by this pull request
@fjwxzde fjwxzde added DEV-自动检查错误 未通过自动检查 and removed DEV-测试通过 通过测试 labels Feb 7, 2025
@DuckDuckStudio
Copy link
Owner Author

Okay, I will try to fix it.

@DuckDuckStudio DuckDuckStudio marked this pull request as draft February 7, 2025 12:42
@DuckDuckStudio DuckDuckStudio marked this pull request as ready for review February 7, 2025 12:59
@fjwxzde fjwxzde added DEV-未经测试 未经测试或无法测试的项 and removed DEV-自动检查错误 未通过自动检查 labels Feb 7, 2025
@fjwxzde
Copy link
Collaborator

fjwxzde commented Feb 7, 2025

你测试过了吗?

fjwxzde

This comment was marked as outdated.

@fjwxzde fjwxzde marked this pull request as draft February 7, 2025 13:10
@DuckDuckStudio DuckDuckStudio added DEV-测试错误 未通过测试 and removed DEV-未经测试 未经测试或无法测试的项 labels Feb 7, 2025
@DuckDuckStudio DuckDuckStudio added DEV-测试通过 通过测试 and removed DEV-测试错误 未通过测试 labels Feb 7, 2025
@DuckDuckStudio DuckDuckStudio removed their assignment Feb 7, 2025
@DuckDuckStudio
Copy link
Owner Author

@DuckDuckStudio DuckDuckStudio marked this pull request as ready for review February 7, 2025 13:49
Copy link
Collaborator

@fjwxzde fjwxzde left a comment

Choose a reason for hiding this comment

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

LGTM

@DuckDuckStudio DuckDuckStudio merged commit aa8ffe8 into main Feb 7, 2025
2 checks passed
@DuckDuckStudio DuckDuckStudio deleted the develop branch February 7, 2025 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DEV-开发分支合并 开发分支的合并PR DEV-测试通过 通过测试
Projects
Status: 已完成
Development

Successfully merging this pull request may close these issues.

[Feat]: 自定义更新提交撰写者 [Feat]: 自动关闭过时的更新请求
2 participants