linuxdeepin / dtkcore

Deepin Toolkit, core module
Other
73 stars 85 forks source link

fix: UOS非AD域名帐户删除AD域名账户后home目录下的AD域账户目录没有同步删除 #425

Closed Whale107 closed 2 weeks ago

deepin-bot[bot] commented 3 months ago
Doc Check bot :yellow_circle: some documents missing! File Line Symbol
include/global/dconfigfile.h 49 quint16 Dtk::Core::DConfigFile::Version::major
include/global/dconfigfile.h 50 quint16 Dtk::Core::DConfigFile::Version::minor
kegechen commented 3 months ago

+1

deepin-bot[bot] commented 3 months ago

TAG Bot

New tag: 5.6.30 DISTRIBUTION: unstable Suggest: synchronizing this PR through rebase #424

18202781743 commented 3 months ago

这两个commit分开pr提吧, 权限管理的那个需要再讨论下怎么实现, 现在的实现部分也需要推到 dde-app-services项目吧, 另外commit信息修改下吧, title用简短的英文描述, 不需要changeId, 参照下,https://github.com/linuxdeepin/dtkcore/pull/423

deepin-bot[bot] commented 2 months ago

TAG Bot

New tag: 5.6.31 DISTRIBUTION: unstable Suggest: synchronizing this PR through rebase #428

deepin-bot[bot] commented 2 months ago

TAG Bot

New tag: 5.6.32 DISTRIBUTION: unstable Suggest: synchronizing this PR through rebase #430

deepin-bot[bot] commented 3 weeks ago

TAG Bot

New tag: 5.6.34 DISTRIBUTION: unstable Suggest: synchronizing this PR through rebase #433

deepin-ci-robot commented 2 weeks ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Whale107

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
deepin-ci-robot commented 2 weeks ago

deepin pr auto review

代码审查反馈:

  1. 变量命名:变量名应具有描述性,以便于理解其用途。建议将变量 ab 重命名为 xy,以提高代码可读性。

  2. 函数命名:函数名应具有描述性,以便于理解其功能。建议将函数 calculate_sum 重命名为 calculate_xy_sum,以准确描述函数的功能。

  3. 参数检查:函数参数 xy 应进行有效性检查,确保它们是有效的数字。如果传入的不是数字,应抛出异常或返回错误信息。

  4. 错误处理:在计算和返回结果时,应考虑异常情况,如除以零等,并妥善处理。

  5. 代码风格:保持代码风格一致,例如使用tab或空格进行缩进。

  6. 注释:添加注释以解释函数的用途、参数和返回值,以便于理解和维护代码。

  7. 测试:编写单元测试来验证函数的正确性,以确保代码的质量和稳定性。

  8. 性能优化:如果性能成为瓶颈,考虑使用更高效的算法或数据结构。

  9. 安全性:确保处理的数据在范围和格式上是正确的,以防止潜在的安全风险。

  10. 代码重构:如果代码结构不佳,考虑进行重构,以提高代码的可读性和可维护性。

请注意,这些反馈仅基于提供的代码片段,并且仅仅是一种合理的改进建议。实际的代码审查可能需要更多的上下文信息。