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

bump to 0.7.2 #317

Merged
merged 2 commits into from
Jun 13, 2024
Merged

bump to 0.7.2 #317

merged 2 commits into from
Jun 13, 2024

Conversation

kegechen
Copy link
Contributor

No description provided.

mhduiy and others added 2 commits June 13, 2024 10:25
The refresh slot function was not successfully connected

Issue: linuxdeepin/developer-center#8968
release 0.7.2

Log: bump version to 0.7.2
@kegechen
Copy link
Contributor Author

/integrate

@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • AppsModel::AppsModel构造函数中新增了m_tmUpdateCache变量,但未看到对应的定时器启动和连接操作。
  • 移除了m_tmUpdateCache的实例化代码,但在构造函数中又重新设置了定时器。
  • 定时器m_tmUpdateCache的间隔设置从1000毫秒变更为1秒,可能影响应用程序的更新频率。

是否建议立即修改:

建议的修改:

  • 确认m_tmUpdateCache的用途,并确保定时器的使用符合设计要求。
  • 如果定时器是必要的,建议在代码中添加相应的注释说明原因。
  • 考虑将定时器的间隔设置保持为1000毫秒,以避免不必要的性能开销。

Copy link

TAG Bot

TAG: 0.7.2
EXISTED: no
DISTRIBUTION: unstable

Copy link

AutoIntegrationPr Bot
auto integrate with pr url: deepin-community/Repository-Integration#1588
PrNumber: 1588
PrBranch: auto-integration-9492715959

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: kegechen, zsien

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

@kegechen kegechen merged commit 083246b into linuxdeepin:release/rc2 Jun 13, 2024
10 of 11 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.

4 participants