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

build: add StudyFolderOrganizer-GUI #2173

Merged
merged 1 commit into from
May 29, 2024
Merged

Conversation

wjyrich
Copy link
Contributor

@wjyrich wjyrich commented May 13, 2024

Mange your study folders in a easy and time saving way. A easy to use graphical app. GUI Version of the StudyFolderOrganizer console app

Log: add software name--StudyFolderOrganizer-GUI
StudyFolderOrganizer-GUI

@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • patches/0001-install.patch文件中,存在代码注释被删除的情况,可能会导致理解上的困难。建议保留或替换被删除的注释。
  • StudyFolderOrganizer-GUI.pro文件中,target.path的赋值使用了拼写错误($$PREFIX应该是${PREFIX}),应该立即修正。
  • desktop.pathicons.path中的$$PREFIX应该使用正确的环境变量替换,以避免潜在的配置错误。
  • icons.files中使用了png/Main App Icon@4x.png,但是对应的文件名在res/logo/目录下可能不存在,需要确认文件路径的正确性。
  • VERSION变量在StudyFolderOrganizer-GUI.pro文件中被赋值,但是没有上下文说明其用途,建议添加注释说明。

是否建议立即修改:

Mange your study folders in a easy and time saving way. A easy to use graphical app. GUI Version of the StudyFolderOrganizer console app

Log: add software name--StudyFolderOrganizer-GUI
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: kamiyadm, wjyrich

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

@kamiyadm kamiyadm merged commit 9b46d6e into linuxdeepin:master May 29, 2024
5 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.

None yet

3 participants