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

fix: remove invalid url #288

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

wyu71
Copy link
Contributor

@wyu71 wyu71 commented Dec 20, 2024

remove invalid url

Log: remove invalid url

remove invalid  url

Log: remove invalid  url
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. 文档链接的移除

    • README.mdREADME.zh_CN.md 文件中,文档链接部分被移除了。如果这些文档链接对于用户来说是有用的,建议重新考虑是否需要保留这些链接,或者提供替代的文档资源。
  2. 文件格式一致性

    • README.mdREADME.zh_CN.md 文件在格式上存在不一致。例如,README.md 文件中的标题使用的是英文,而 README.zh_CN.md 文件中的标题使用的是中文。建议统一使用一种语言来保持文档的一致性。
  3. 帮助部分的内容

    • README.zh_CN.md 文件中,帮助部分的描述不够清晰。建议提供更详细的信息,比如如何通过官方论坛寻求帮助的具体步骤。
  4. 翻译的一致性

    • README.zh_CN.md 文件中,"命令行启动 deepin-album" 的翻译可能不够准确。建议检查并确保翻译的一致性和准确性。
  5. 链接的有效性

    • 确保所有提供的链接(如文档链接和帮助论坛链接)都是有效的,并且可以正常访问。如果链接失效,用户将无法获取所需的信息。
  6. 代码风格

    • 文件中的链接格式应该保持一致,例如,所有的链接都应该使用相同的格式(如使用方括号和圆括号)。

综上所述,建议重新考虑文档链接的移除,保持文件格式的一致性,并提供更详细和准确的帮助信息,同时确保所有链接的有效性。

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: pengfeixx, wyu71

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@wyu71 wyu71 merged commit b03ae10 into linuxdeepin:master-qml-reform Dec 20, 2024
14 checks passed
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