Skip to content

Conversation

OskarStark
Copy link
Contributor

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

Following the codebase convention to use $this->assert* instead of self::assert* for better consistency across all test files

@carsonbot carsonbot changed the title fix: Replace self::assert* with $this->assert* in test files fix: Replace self::assert* with $this->assert* in test files Sep 3, 2025
@OskarStark OskarStark added Platform Issues & PRs about the AI Platform component Agent Issues & PRs about the AI Agent component Demo Issues & PRs about the demo application labels Sep 3, 2025
@carsonbot carsonbot changed the title fix: Replace self::assert* with $this->assert* in test files [Agent][Demo][Platform] fix: Replace self::assert* with $this->assert* in test files Sep 3, 2025
@OskarStark OskarStark changed the title [Agent][Demo][Platform] fix: Replace self::assert* with $this->assert* in test files [Agent][Demo][Platform] Replace self::assert* with $this->assert* in test files Sep 3, 2025
@OskarStark OskarStark force-pushed the fix/prefer-this-assert branch from aacd1d7 to 20f3d0c Compare September 3, 2025 07:58
@OskarStark OskarStark merged commit 2decdd0 into symfony:main Sep 3, 2025
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 Demo Issues & PRs about the demo application 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