Skip to content

Conversation

OskarStark
Copy link
Contributor

Q A
Bug fix? no
New feature? no
Docs? no
Issues --
License MIT

Changed multi-line #[\SensitiveParameter] attributes to single-line format for better readability and consistency across the codebase.

Changed multi-line #[\SensitiveParameter] attributes to single-line format
for better readability and consistency across the codebase.
@carsonbot carsonbot changed the title Fix #[\SensitiveParameter] formatting to single line Fix #[\SensitiveParameter] formatting to single line Sep 3, 2025
Copy link
Member

@chr-hertel chr-hertel left a comment

Choose a reason for hiding this comment

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

for better readability

Subjective, your honor! :D

no hard feelings tho - not sure what's the "standard" here

@OskarStark OskarStark added Platform Issues & PRs about the AI Platform component Agent Issues & PRs about the AI Agent component labels Sep 3, 2025
@carsonbot carsonbot changed the title Fix #[\SensitiveParameter] formatting to single line [Agent][Platform] Fix #[\SensitiveParameter] formatting to single line Sep 3, 2025
@OskarStark OskarStark merged commit 56f51eb into symfony:main Sep 3, 2025
7 checks passed
@OskarStark OskarStark deleted the fix-sensitive-parameter-formatting branch September 3, 2025 07:45
@OskarStark
Copy link
Contributor Author

no hard feelings tho - not sure what's the "standard" here

Symfony standard is online for this attribute in Mailer/Notifer, hence the PR 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Agent Issues & PRs about the AI Agent component Platform Issues & PRs about the AI Platform component Status: Reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants