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: Fixed an issue where cmake could not be found when the project was opened #891

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

Kakueeen
Copy link
Contributor

Modify prompt information

Log: fix issue

…as opened

Modify prompt information

Log: fix issue
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • 在构造函数中移除了不必要的空行,简化了代码结构。
  • 修改了错误消息的格式,提供了更详细的解决方案,有助于用户解决问题。

是否建议立即修改:

  • 否,这次提交的修改主要是风格和用户友好性的改进,没有发现需要立即修改的关键问题。但是,如果有更多的上下文,比如整个项目的代码风格和最佳实践,可能需要进一步的审查来确保整体代码质量的提高。

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deepin-mozart, Kakueeen

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

The pull request process is described 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-mozart deepin-mozart merged commit d52cd0a into linuxdeepin:master Aug 20, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants