Skip to content

Conversation

@JWWTSL
Copy link

@JWWTSL JWWTSL commented Jun 19, 2025

log: Updating transifex platform related configuration files

Summary by Sourcery

Chores:

  • Revise .tx/config with updated Transifex resource settings

@sourcery-ai
Copy link

sourcery-ai bot commented Jun 19, 2025

Reviewer's Guide

This PR refactors the Transifex configuration (.tx/config) to align project and resource identifiers with the new platform setup and updates file‐matching patterns for source and translated content.

File-Level Changes

Change Details Files
Updated Transifex host and authentication placeholders
  • Replaced hardcoded API URL with TX_HOST environment variable
  • Standardized auth token placeholder format
.tx/config
Revised project and resource slugs
  • Renamed project_slug to reflect new repository name
  • Updated resource_slug entries for UI and documentation strings
.tx/config
Adjusted file mapping patterns
  • Expanded file_globs to include new source directories
  • Changed source_file and target_file paths for consistency
  • Added nesting for locale directories
.tx/config
Removed deprecated resource definitions
  • Deleted unused ‘legacy_text’ resource block
  • Cleaned up obsolete language mappings
.tx/config

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

Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey @JWWTSL - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. 文件路径和语言代码的统一

    • 在多个配置文件中,源文件路径和语言代码被统一修改为 translations/deepin-compressor.tsen_US。这有助于保持代码的一致性,但需要确认这些更改是否与项目的其他部分兼容。
  2. 重复的配置项

    • deepin-archive-manager.deepin-archive-manager-conf-singleextract-tsdeepin-archive-manager.deepin-archive-manager-conf-multiextract-ts 中,文件过滤器和源文件路径被重复定义。建议检查这些配置项是否确实需要,或者是否可以合并以减少冗余。
  3. 新增的配置项

    • 新增了三个配置项,分别对应 multicompressmultiextractsingleextract。需要确认这些新增配置项是否确实需要,并且与现有配置项没有冲突。
  4. 缺少的 preTranslate_file

    • 在一些配置项中,preTranslate_file 被移除或未定义。如果这些文件确实需要预翻译,应该重新添加这些路径。
  5. 项目引用的版本号

    • 在每个配置项中,项目引用的版本号(r)被修改。需要确认这些版本号是否正确,并且与项目的实际版本一致。
  6. 配置文件格式

    • 配置文件格式保持一致,没有明显的格式错误。但是,建议检查是否有遗漏的逗号或其他格式问题。
  7. 代码注释

    • 考虑为新增的配置项添加注释,以便其他开发者理解这些更改的目的和影响。

总体来说,这些更改看起来是为了统一和简化翻译配置文件。需要确保这些更改不会影响项目的其他部分,并且与项目的整体目标一致。

@JWWTSL JWWTSL closed this Jun 26, 2025
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: JWWTSL

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

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