Skip to content

fix: add unix_mode when copy_raw_file - #981

Merged
Its-Just-Nans merged 4 commits into
masterfrom
fix-copy-raw-touch-file
Sep 8, 2026
Merged

Its-Just-Nans merged 4 commits into
masterfrom
fix-copy-raw-touch-file

Conversation

@Its-Just-Nans

Copy link
Copy Markdown
Member

@amazon-q-developer amazon-q-developer Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look good. The fix properly clears external_attributes before setting new Unix permissions in raw_copy_file_touch, preventing potential conflicts when explicitly providing unix_mode. The accompanying test adequately verifies this behavior.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

@Its-Just-Nans
Its-Just-Nans added this pull request to the merge queue Sep 8, 2026
Merged via the queue into master with commit 386788c Sep 8, 2026
134 checks passed
@Its-Just-Nans
Its-Just-Nans deleted the fix-copy-raw-touch-file branch September 8, 2026 23:54
@Pr0methean Pr0methean mentioned this pull request Sep 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant