Skip to content

Conversation

@dengzhongyuan365-dev
Copy link
Contributor

@dengzhongyuan365-dev dengzhongyuan365-dev commented Jun 24, 2025

fix: Remove spaces in archive manager titles for consistency

  • Updated the titles in the archive manager markdown files to remove spaces before the pipe symbol for a uniform appearance across different language versions.

log: Remove spaces in archive manager titles for consistency

bug: https://pms.uniontech.com/bug-view-321609.html

Summary by Sourcery

Remove spaces before the pipe symbol in Archive Manager titles across English and Chinese markdown files for consistent formatting and update the Debian changelog accordingly.

Bug Fixes:

  • Unify Archive Manager title formatting by removing spaces before the pipe symbol in en_US, zh_HK, and zh_TW markdown files.

Build:

  • Update debian/changelog entry to document the title formatting fix.

- Updated the titles in the archive manager markdown files to remove spaces before the pipe symbol for a uniform appearance across different language versions.

log: Remove spaces in archive manager titles for consistency

bug: https://pms.uniontech.com/bug-view-321609.html
 - update version to 6.5.8

 log: update version to 6.5.8
@sourcery-ai
Copy link

sourcery-ai bot commented Jun 24, 2025

Reviewer's Guide

Removed extraneous spaces before the pipe delimiter in the archive manager titles across all localized markdown files and updated the Debian changelog to log the formatting fix.

File-Level Changes

Change Details Files
Normalized title formatting by removing spaces before the pipe separator in all archive-manager markdown headers
  • Deleted space before “
” in English title
  • Deleted space before “
  • Updated Debian changelog to record the formatting correction
    • Added fix entry for removing spaces in archive manager titles
    • Referenced bug tracker ID for consistency
    debian/changelog

    Tips and commands

    Interacting with Sourcery

    • Trigger a new review: Comment @sourcery-ai review on the pull request.
    • Continue discussions: Reply directly to Sourcery's review comments.
    • Generate a GitHub issue from a review comment: Ask Sourcery to create an
      issue from a review comment by replying to it. You can also reply to a
      review comment with @sourcery-ai issue to create an issue from it.
    • Generate a pull request title: Write @sourcery-ai anywhere in the pull
      request title to generate a title at any time. You can also comment
      @sourcery-ai title on the pull request to (re-)generate the title at any time.
    • Generate a pull request summary: Write @sourcery-ai summary anywhere in
      the pull request body to generate a PR summary at any time exactly where you
      want it. You can also comment @sourcery-ai summary on the pull request to
      (re-)generate the summary at any time.
    • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
      request to (re-)generate the reviewer's guide at any time.
    • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
      pull request to resolve all Sourcery comments. Useful if you've already
      addressed all the comments and don't want to see them anymore.
    • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
      request to dismiss all existing Sourcery reviews. Especially useful if you
      want to start fresh with a new review - don't forget to comment
      @sourcery-ai review to trigger a new review!

    Customizing Your Experience

    Access your dashboard to:

    • Enable or disable review features such as the Sourcery-generated pull request
      summary, the reviewer's guide, and others.
    • Change the review language.
    • Add, remove or edit custom review instructions.
    • Adjust other review settings.

    Getting Help

    @deepin-ci-robot
    Copy link

    deepin pr auto review

    代码审查意见:

    1. 文件格式一致性

      • debian/changelog文件中,提交信息中使用了* fix:,建议使用一致的格式,例如- fix:+ fix:,以保持提交信息的清晰和一致性。
    2. 文件命名和内容一致性

      • src/assets/deepin-compressor/archive-manager/en_US/archive-manager.mdsrc/assets/deepin-compressor/archive-manager/zh_HK/archive-manager.mdsrc/assets/deepin-compressor/archive-manager/zh_TW/archive-manager.md文件中,标题中的|符号被移除,这可能会影响文档的格式和一致性。建议确认这一改动是否符合设计规范,并确保所有语言版本的文档都进行了相应的更新。
    3. 国际化(i18n)

      • 如果移除|符号是为了保持界面的一致性,建议检查其他语言版本的文档,确保它们也进行了相应的更新,以保持多语言版本的界面一致性。
    4. 提交信息

      • debian/changelog文件中,提交信息中包含了日期和时间,建议使用标准的日期格式,例如YYYY-MM-DD,以提高可读性。
    5. 代码注释

      • 考虑在src/assets/deepin-compressor/archive-manager/en_US/archive-manager.mdsrc/assets/deepin-compressor/archive-manager/zh_HK/archive-manager.mdsrc/assets/deepin-compressor/archive-manager/zh_TW/archive-manager.md文件中添加注释,说明为什么移除|符号,以及这一改动如何影响文档的布局和可读性。

    总体来说,这些改动看起来是为了保持界面的一致性,但需要确保所有相关的文档和语言版本都进行了相应的更新和测试,以避免潜在的问题。

    @github-actions
    Copy link

    TAG Bot

    TAG: 6.5.8
    EXISTED: no
    DISTRIBUTION: unstable

    @lzwind lzwind merged commit 9d54457 into linuxdeepin:develop/snipe Jun 24, 2025
    15 checks passed
    @deepin-ci-robot
    Copy link

    [APPROVALNOTIFIER] This PR is NOT APPROVED

    This pull-request has been approved by: dengzhongyuan365-dev, lzwind

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

    Details 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

    @lzwind
    Copy link
    Contributor

    lzwind commented Jun 24, 2025

    /topic cd-20250623

    @deepin-ci-robot
    Copy link

    Add topic: cd-20250623 successed.

    @lzwind
    Copy link
    Contributor

    lzwind commented Jun 24, 2025

    /integr-topic cd-20250623

    @deepin-ci-robot
    Copy link

    Integrated with pr deepin-community/Repository-Integration#2943

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

    Labels

    None yet

    Projects

    Status: No status

    Development

    Successfully merging this pull request may close these issues.

    3 participants