Skip to content

Conversation

Guikingone
Copy link
Contributor

Q A
Bug fix? no
New feature? no
Docs? yes
Issues Fix none
License MIT

Hi 👋🏻

Here's the rag example for Clickhouse as discussed in #335

@carsonbot carsonbot added Status: Needs Review Run examples Issues & PRs about the example scripts labels Sep 1, 2025
@OskarStark OskarStark changed the title [Examples] Add Clickhouse rag [Examples][ClickHouse] Add RAG example Sep 1, 2025
@OskarStark
Copy link
Contributor

Thank you @Guikingone.

@OskarStark OskarStark merged commit ecbe6e5 into symfony:main Sep 1, 2025
7 checks passed
@Guikingone Guikingone deleted the examples/clickhouse branch September 2, 2025 07:03
derrabus pushed a commit to derrabus/symfony-ai that referenced this pull request Sep 12, 2025
…#389)

## Summary

This PR updates the PHPStan configuration to use an error identifier
instead of a regex pattern for ignoring array type errors in tests.

## Changes

- Replace regex pattern  with identifier 

## Benefits

Using error identifiers is the recommended approach because:
- **More stable**: Identifiers don't change across PHPStan versions,
unlike error message text
- **Cleaner**: Easier to read and understand the intent
- **Maintainable**: Less prone to breaking if error message wording
changes

## Testing

PHPStan analysis passes with the new configuration.
derrabus pushed a commit to derrabus/symfony-ai that referenced this pull request Sep 12, 2025
…ex pattern (OskarStark)

This PR was merged into the main branch.

Discussion
----------

[Agent] Use PHPStan error identifier instead of regex pattern

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

Cherry picking php-llm/llm-chain#389

Commits
-------

611206b chore: use PHPStan error identifier instead of regex pattern (symfony#389)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Run examples Issues & PRs about the example scripts Status: Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants