Skip to content

refactor: cleanup permissions related code - #961

Merged
Its-Just-Nans merged 3 commits into
masterfrom
cleanup-permissions
Aug 30, 2026
Merged

Its-Just-Nans merged 3 commits into
masterfrom
cleanup-permissions

Conversation

@Its-Just-Nans

@Its-Just-Nans Its-Just-Nans commented Aug 30, 2026 •

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 in this PR improve code organization and correctness without introducing defects. The refactoring moves permission constants to the appropriate module, fixes variable scoping in src/types.rs, and enhances test coverage. All changes function correctly and maintain backward compatibility.


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.

Comment thread tests/end_to_end.rs Dismissed
@Its-Just-Nans
Its-Just-Nans added this pull request to the merge queue Aug 30, 2026
Merged via the queue into master with commit 803adde Aug 30, 2026
135 checks passed
@Its-Just-Nans
Its-Just-Nans deleted the cleanup-permissions branch August 30, 2026 17:33
@Pr0methean Pr0methean mentioned this pull request Aug 30, 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.

2 participants