fix: Update .gitignore to include additional build and release artifacts #303
Merged
deepin-bot[bot] merged 2 commits intolinuxdeepin:release/eaglefrom Jul 28, 2025
Merged
Conversation
Added entries for release, obj-x86_64-linux-gnu, and various debian-related files to the .gitignore to prevent unnecessary files from being tracked in the repository. Log: Update .gitignore to include additional build and release artifacts Bug: no
Updated the logic for constructing the archive path to handle long folder names, ensuring they do not exceed system limits. This includes trimming folder names when necessary and improving the overall robustness of the path handling. Log: Improve folder name handling in archive path construction Bug: https://pms.uniontech.com/bug-view-326399.html
deepin pr auto review代码审查意见: .gitignore 文件
mainwindow.cpp 文件
总体来说,代码的改动看起来是为了处理长文件夹名称的问题,并且保持了代码的整洁性。但是,需要确保这些改动不会影响现有的功能,并且已经通过了充分的测试。 |
max-lvs
approved these changes
Jul 28, 2025
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: max-lvs, pppanghu77 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Contributor
Author
|
/merge |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fix: Improve folder name handling in archive path construction
Updated the logic for constructing the archive path to handle long folder names, ensuring they do not exceed system limits. This includes trimming folder names when necessary and improving the overall robustness of the path handling.
Log: Improve folder name handling in archive path construction
Bug: https://pms.uniontech.com/bug-view-326399.html